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

Prevent the use of the list address as owner or moderator #297

Closed
ldidry opened this issue May 15, 2018 · 7 comments
Closed

Prevent the use of the list address as owner or moderator #297

ldidry opened this issue May 15, 2018 · 7 comments
Labels
Milestone

Comments

@ldidry
Copy link
Contributor

ldidry commented May 15, 2018

We got the case where the list was moderated for everyone and the owner was the list address, causing a mail loop.

I think this should not happen.

Seen on 6.2.16.

@ldidry ldidry added the bug label May 15, 2018
@ikedas
Copy link
Member

ikedas commented Jun 21, 2018

cf. #345. This seems fixed on 6.2.17.

@sivertkh
Copy link
Member

Looks like this is still a problem on 6.2.32.

Moderated list. One of the administrators used the "Changing your email address" feature to changed there email to a list address.
When an email was delivered to the list, a moderation email was sent back to the list starting a moderatron loop.

@ikedas
Copy link
Member

ikedas commented Jul 4, 2018

@sivertkh, I understand. #345 doesn't deal with list address.

Are there any use cases to add list address as a moderator (or owner, if moderators are not defined)?

@ikedas
Copy link
Member

ikedas commented Aug 9, 2018

Hi folks,

Ok, there seem no use cases to add list address as a moderator/owner.

Please check PR above.

@ikedas ikedas added this to the 6.2.36 milestone Aug 9, 2018
@ldidry
Copy link
Contributor Author

ldidry commented Aug 9, 2018

Seems good to me.

ikedas added a commit that referenced this issue Aug 16, 2018
Prevent the use of the list address as owner or moderator #297
@ikedas
Copy link
Member

ikedas commented Aug 16, 2018

Merged. Thanks for reporting bug!
Closing.

@ikedas ikedas closed this as completed Aug 16, 2018
@ldidry
Copy link
Contributor Author

ldidry commented Aug 16, 2018

Patched and tested, works well 👍

Thank you.

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

No branches or pull requests

3 participants