Policy extension: Allow servers to advertize clear policy information and require consent#564
Open
ValwareIRC wants to merge 3 commits intoircv3:masterfrom
Open
Policy extension: Allow servers to advertize clear policy information and require consent#564ValwareIRC wants to merge 3 commits intoircv3:masterfrom
ValwareIRC wants to merge 3 commits intoircv3:masterfrom
Conversation
kylef
reviewed
Jun 11, 2025
kylef
reviewed
Jun 12, 2025
| :irc.example.com NOTE POLICY PRIVACY 2 :Data is retained for 30 days. | ||
| :irc.example.com NOTE POLICY PRIVACY * :End of Privacy & Data Use. | ||
|
|
||
| :irc.example.com NOTE POLICY * :To accept, click “Accept” or type /CONSENT |
Contributor
There was a problem hiding this comment.
click “Accept” may be making a few assumptions about the interface the client presents, which the server won't know (client may use different wording, be CLI without ability to "click" or have localisation).
Perhaps this line should be omitted and clients are encouraged to provide a clear way to accept the policies.
For example, a GUI application may present a modal view which displays the text and has an button to agree or disagree. They may not have an input bar to type "/CONSENT. At-least that is how I'd likely have designed this as a client developer
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Edit: Changed the language to reflect just a "policy" instead of a "privacy policy" specifically. this is so that servers can show all policies. Updated language in NOTE POLICY responses to separate different types of policy to make it easier for a UI implementing this to make it beautiful/improve UX