-
-
Notifications
You must be signed in to change notification settings - Fork 409
docs: config/auth: clarify auth_username, etc #2105
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
Conversation
|
For 8.0:
I don't think it's worth adding |
dgw
left a 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 only thing I don't understand about this patch is the addition of "allowed or" in auth_username's description.
Exirel
left a 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.
Happy to see more/better documentation!
|
dgw, I phrased it badly, but the intent was to remove the implication that "it's allowed, and sometimes it's required" since it's never optional. |
dgw
left a 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.
I had already marked these changes as viewed, so I don't know why there wasn't also a (re-)review to go with that. Oh well, squash away.
0e27bd8 to
e792611
Compare
Description
Add an example for having Sopel authenticate to ZNC, inspired by a user in IRC confused by
auth_usernamenot working withauth_method=server.Also corrects/clarifies/creates a few other things:
Checklist
make qa(runsmake qualityandmake test)