Skip to content
This repository has been archived by the owner on Jun 24, 2021. It is now read-only.

Implement the freenode.net/identify-msg vendor cap #356

Closed
wants to merge 1 commit into from

Conversation

edk0
Copy link
Contributor

@edk0 edk0 commented Jul 27, 2020

There's no prior art, so we can change the capability name to something charybdis-y if you have a domain and you want to.

m_identified gets to be a core module because we need to maintain identified status at all times—there's no way to recover it after module downtime. I suspect m_services should get the same treatment, but that'll be another PR, if so.

@kaniini
Copy link
Contributor

kaniini commented Jul 27, 2020

if they are authed to an account, that should be sufficient. what is this identified thing about?

DALnet-style NICK ownership has never been a design concept in charybdis, why should we make it one now?

@kaniini
Copy link
Contributor

kaniini commented Jul 27, 2020

If we were to implement a DALnet-style nick ownership mode, it should be implemented with a user-mode, not this module approach. That allows it to be an extension, which is basically the only thing that is tolerable.

@kaniini
Copy link
Contributor

kaniini commented Jul 27, 2020

Here's how we are going to actually do this.

We are going to add a usermode +r, which can only be set or cleared by services. On nick change, the usermode will automatically clear. This will NOT be a required module to run charybdis, it will be an extension. Freenode will simply enable the extension.

Instead of adding yet another ENCAP, we're going to add ENCAP MODE to allow services to set this user mode. Other ENCAPs should also be replaced by ENCAP MODE.

@kaniini
Copy link
Contributor

kaniini commented Jul 27, 2020

Closing in favor of #357 which is a design that is acceptable to us.

@kaniini kaniini closed this Jul 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants