Skip to content
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

Default Ghost sequence changed to REGAIN #547

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Conversation

casperstorm
Copy link
Member

Fixes #544

@glguy
Copy link

glguy commented Sep 10, 2024

That's better. Looking more closely I'll also note that the logic will typically be unused as ideally users aren't setting the nickname password field because they're using SASL to authenticate. REGAIN and GHOST only need a password when you're not already identified.

@casperstorm casperstorm merged commit dfe3cd1 into main Sep 12, 2024
1 check passed
@casperstorm casperstorm deleted the default-ghost-regain branch September 12, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ghost sequence should default to REGAIN
2 participants