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

Document ERR_INVALIDUSERNAME and ERR_ONLYSERVERSCANCHANGE #217

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

progval
Copy link
Member

@progval progval commented Sep 2, 2023

Sadly, both are in common use:

  • ERR_INVALIDUSERNAME is used by Ergo, InspIRCd, and ircu2/Nefarious/snircd
  • ERR_ONLYSERVERSCANCHANGE is used by Bahamut, Plexus4, and UnrealIRCd

I'll discuss this with server devs to try to resolve the clash.

Sadly, both are in common use:

* ERR_INVALIDUSERNAME is used by Ergo, InspIRCd, and ircu2/Nefarious/snircd
* ERR_ONLYSERVERSCANCHANGE is used by Bahamut, Plexus4, and UnrealIRCd
progval added a commit to progval/sable that referenced this pull request Sep 2, 2023
This used to be valid because '[' is stripped from usernames

This uses the ERR_INVALIDUSERNAME numeric I am about to introduce to
modern.ircdocs.horse: ircdocs/modern-irc#217
progval added a commit to progval/sable that referenced this pull request Sep 2, 2023
This used to be valid because '[' is stripped from usernames

This uses the ERR_INVALIDUSERNAME numeric I am about to introduce to
modern.ircdocs.horse: ircdocs/modern-irc#217
spb pushed a commit to Libera-Chat/sable that referenced this pull request Sep 2, 2023
This used to be valid because '[' is stripped from usernames

This uses the ERR_INVALIDUSERNAME numeric I am about to introduce to
modern.ircdocs.horse: ircdocs/modern-irc#217
@SadieCat
Copy link
Contributor

SadieCat commented Sep 3, 2023

FWIW we use ERR_NOPRIVILEGES (481) in the case that Unreal uses ERR_ONLYSERVERSCANCHANGE for.

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.

None yet

2 participants