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

Check Newsgroups header of cancel messages #52

Closed
rra opened this issue Jul 17, 2021 · 3 comments
Closed

Check Newsgroups header of cancel messages #52

rra opened this issue Jul 17, 2021 · 3 comments
Assignees
Labels
C: innd Related to innd enhancement New feature or request P: low Low priority RFC compliance Related to NNTP standards and the format of Usenet articles
Milestone

Comments

@rra
Copy link
Contributor

rra commented Jul 17, 2021

Reported by eagle on 14 Dec 2008 07:34 UTC
A patch from Christopher Biedl to alter ARTcancelverify to check whether at least one group in the cancel message can be found in article to be canceled.

@rra rra added C: innd Related to innd P: low Low priority T: enhancement labels Jul 17, 2021
@rra rra self-assigned this Jul 17, 2021
@rra
Copy link
Contributor Author

rra commented Jul 17, 2021

Comment by iulius on 14 Dec 2008 19:21 UTC
Maybe this feature should be added to the check already done on Sender: and From: headers. (This patch currently removes that check.)

@rra
Copy link
Contributor Author

rra commented Jul 17, 2021

Comment by iulius on 14 Aug 2009 19:00 UTC
According to RFC 5537 (USEPRO):

Contrary to RFC 1036 (Horton, M. and R. Adams, Standard for interchange of USENET messages, December 1987.), cancel control messages are not required to contain From and Sender header fields matching the target message. This requirement only encouraged cancel issuers to conceal their identity and provided no security.

The check done on Sender: and From: headers can therefore be removed.

@rra rra added this to the 2.5.1 milestone Jul 17, 2021
@rra
Copy link
Contributor Author

rra commented Jul 17, 2021

Comment by iulius on 14 Aug 2009 19:45 UTC
(In [8564]) A patch from Christopher Biedl to alter ARTcancelverify
to check whether at least one group in the cancel message
can be found in the article to be cancelled.

The check for matching Sender: and From: headers is useless
and removed.

close #52

@rra rra added the R: fixed label Jul 17, 2021
@rra rra closed this as completed Jul 17, 2021
@Julien-Elie Julien-Elie assigned Julien-Elie and unassigned rra Nov 4, 2021
@Julien-Elie Julien-Elie added enhancement New feature or request RFC compliance Related to NNTP standards and the format of Usenet articles and removed T: enhancement labels Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: innd Related to innd enhancement New feature or request P: low Low priority RFC compliance Related to NNTP standards and the format of Usenet articles
Development

No branches or pull requests

2 participants