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

Rework of account/channel services integration #1930

Open
3 of 6 tasks
SadieCat opened this issue Oct 5, 2021 · 1 comment
Open
3 of 6 tasks

Rework of account/channel services integration #1930

SadieCat opened this issue Oct 5, 2021 · 1 comment
Labels
enhancement New feature or request
Milestone

Comments

@SadieCat
Copy link
Member

SadieCat commented Oct 5, 2021

Currently services integration is very messy and doesn't allow for operation without services. This is undesirable as it doesn't allow us to implement shiny new things like distributed services.

I propose that in v4 we make the following changes:

  • All of the modes other than channel and user modes r are moved from services_account to new account and channel (working names) modules
  • The services_account module is renamed to services and solely handles services integration going forwards.
  • The servprotect module is merged into the services module.
  • The user-facing parts of the sasl module are moved to the account module and the S2S part is moved to the services module.
  • Support for the IRCv3 account registration spec is implemented in the account module with the S2S part in the services module.
  • A new module (name as yet undecided) is added that allows support for distributed account/channel registration and authentication without a services server.

Additionally, having user and channel modes +r seems unnecessary and their behaviour could very well be converted to user and channel metadata which would free up two valuable mode characters. That's out of scope for this though.

@SadieCat SadieCat added the enhancement New feature or request label Oct 5, 2021
@SadieCat SadieCat added this to the v4.0 milestone Oct 5, 2021
@SadieCat SadieCat changed the title Rework of account services integration Rework of account/channel services integration Oct 5, 2021
@SadieCat
Copy link
Member Author

SadieCat commented Oct 8, 2021

Having checked Anope and asked nicole about Atheme it seems that channel mode +r is not used for anything other than as a user-visible marker by modern services implementations. User mode +r does have a use ("user is logged into their nick") but this can be very trivially converted to metadata which would actually be a significant improvement as it removes the user facing -r+r spam when a user switches to their alt nick.

@inspircd inspircd deleted a comment from awfulcooking Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

1 participant