Skip to content

fix #1933#1936

Merged
slingamn merged 2 commits intoergochat:masterfrom
slingamn:nick_empty
Apr 25, 2022
Merged

fix #1933#1936
slingamn merged 2 commits intoergochat:masterfrom
slingamn:nick_empty

Conversation

@slingamn
Copy link
Copy Markdown
Member

No description provided.

If the nickname must equal the account name (because always-on or
force-nick-equals-account), the correct error response to an empty
or otherwise invalid nickname is the usual "You must use your account
name as your nickname".
`NICK :` pre-registration needs to be special-cased to immediately
send ERR_NONICKNAMEGIVEN (unlike erroneous nonempty nicknames,
which are processed when registration is complete)
@slingamn slingamn added this to the v2.10 milestone Apr 24, 2022
@slingamn slingamn added the bug label Apr 24, 2022
@slingamn slingamn merged commit 5c7df07 into ergochat:master Apr 25, 2022
@slingamn slingamn deleted the nick_empty branch January 28, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant