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

/wws/renewpasswd can generate nuisance messages #1076

Closed
dpc22 opened this issue Jan 11, 2021 · 4 comments · Fixed by #1082
Closed

/wws/renewpasswd can generate nuisance messages #1076

dpc22 opened this issue Jan 11, 2021 · 4 comments · Fixed by #1082
Milestone

Comments

@dpc22
Copy link
Contributor

dpc22 commented Jan 11, 2021

Expected Behaviour

Automated scripts should not be able to send email messages to random recipients using the wwsympa .../renewpasswd link.

Current Behaviour

This is possible. While the content is fixed (see attached example) it has the potential to cause nuisance or confusion.

Example.txt

Possible Solution

Mailman can use a CAPCHA to protect the equivalent function. Alternately some form of rate limit on renewpasswd requests?

I found #492, but that doesn't protect against scripts attacking renewpasswd directly

Context

I received a couple of hundred bounce messages over the weekend from attempts to send password renewals to the invalid address sample@email.tst. There were also lots of SQL insertion attacks from the same source IP address in a very short interval, so obviously some form of script.

@dpc22 dpc22 changed the title /wws/renewpasswd can generate nuisance messags /wws/renewpasswd can generate nuisance messages Jan 11, 2021
@racke
Copy link
Contributor

racke commented Jan 11, 2021

Another alternative to CAPTCHAS: https://advent.perldancer.org/2018/21

@ikedas
Copy link
Member

ikedas commented Jan 14, 2021

Hi @dpc22 ,

Could you please apply this patch to check if it will get rid of attackers?

@dpc22
Copy link
Contributor Author

dpc22 commented Jan 18, 2021

Yes, I think that will work nicely. Thank you.

@ikedas ikedas added this to the 6.2.62 milestone Jan 20, 2021
@ikedas
Copy link
Member

ikedas commented Jan 20, 2021

@dpc22 , thanks for reporting bug and confirming fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants