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

XEP-0377: Add report processing opt-in #1428

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

guusdk
Copy link
Contributor

@guusdk guusdk commented Mar 11, 2025

Introduces an additional feature that allows end-users to opt-in to processing of submitted reports.

Changes like these were discussed on the XSF's 'standards' mailing list, as archived at https://mail.jabber.org/hyperkitty/list/[email protected]/thread/RDLWOMBMT24DCH54O2HEV2S3WW67FETE/

@guusdk guusdk requested a review from SamWhited March 11, 2025 12:36
@github-actions github-actions bot added the Needs Author The XEP is experimental and the PR was not made by the author. The author needs to acknowledge it. label Mar 11, 2025
@iNPUTmice
Copy link
Contributor

Pinging @SamWhited for approval.

@SamWhited
Copy link
Member

I generally think we should design an actual reporting system, figure out what options we want, etc. and then figure out what protocol features we need to support that and suspect that this may not actually provide us with a good system to handle how reports get forwarded around, but I'm also not really involved anymore so I certainly won't stand in the way. I'm happy to defer to what Guus wants here.

@guusdk
Copy link
Contributor Author

guusdk commented Mar 11, 2025 via email

@iNPUTmice
Copy link
Contributor

Hi @SamWhited can I take what you said as you are being OK with @guusdk taking over as co-author? (Granting him the right to make and/or approve further changes?)

@guusdk Would you want to become author? In that case (and pending ack from @SamWhited) can you add yourself to the list of authors?

@guusdk
Copy link
Contributor Author

guusdk commented Apr 9, 2025

Eh, that wasn't my original intention, but sure, I can be co-author.

@SamWhited
Copy link
Member

That's fine with me; thanks all!

guusdk added 3 commits April 9, 2025 15:48
Introduces an additional feature that allows end-users to opt-in to processing of submitted reports.

Changes like these were discussed on the XSF's 'standards' mailing list, as archived at https://mail.jabber.org/hyperkitty/list/[email protected]/thread/RDLWOMBMT24DCH54O2HEV2S3WW67FETE/
@guusdk guusdk force-pushed the XEP-0377_Spam-reporting-processing branch from 0661a29 to 6394dab Compare April 9, 2025 13:58
@guusdk
Copy link
Contributor Author

guusdk commented Apr 9, 2025

Just now, this PR:

  • was rebased to the current HEAD of the Master branch (to allow for better validation of multiple XEP-files (which is a recently applied improvement to CI);
  • received a new commit in which the author element that refers to Guus in all preexisting XEPs is replaced by an XML entity;
  • received another commit that added Guus as a co-author to XEP-0377 (using the new XML entity), and updated the remarks in the revision to reflect this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Author The XEP is experimental and the PR was not made by the author. The author needs to acknowledge it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants