-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a no_auth_user #1363
Add a no_auth_user #1363
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, LGTM, just a note about checking of misconfiguration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This configuration allows to refer to a configured user to be used when the connection provides no credentials. Signed-off-by: Matthias Hanel <mh@synadia.com>
423cef5
to
b074c94
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Just minor question.
@@ -410,8 +410,10 @@ func validateOptions(o *Options) error { | |||
if err := validateLeafNode(o); err != nil { | |||
return err | |||
} | |||
// Check that gateway is properly configured. Returns no error |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did we lose this comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The gateway one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@derekcollison Good find. Mind if I add it back in with the change that calls validateOptions after reload?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure.
Added a no_auth_user as discussed.
To simplify error checking, I decided to do a post parsing error pass.
Reason being, we have multiple sections where users can be configured (multiple accounts as well authorization block).
Signed-off-by: Matthias Hanel mh@synadia.com