You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
gorums: peer configuration errors repeat the gorums: prefix #369
NewServer and NewLocalServers wrap peer configuration errors in gorums: invalid peer configuration: ….
The wrapped errors already start with gorums: , so the message repeats the prefix:
The verifier errors in #359 do not have this problem: stream.VerificationError is the only layer that adds the prefix.
Options
Drop the wrapper: NewServer panics with the error itself, and NewLocalServers returns it. The inner errors already start with gorums: and name the problem.
Keep the wrapper and remove the prefix from the inner errors. This does not work for the node source errors, because NewConfig returns them without a wrapper.
Option 1 is the smaller change.
Done when no configuration error repeats gorums:, and the tests pin the single-prefix text.
Activity
gorums: drop the duplicate prefix from peer configuration errors