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

Use PHP's e-mail validation filter to determine if list name is valid #28

Closed

Conversation

kienanstewart
Copy link
Contributor

Fixes #27

@camlafit
Copy link
Contributor

@L-bryan
Copy link

L-bryan commented May 8, 2019

just for be sure , the '-' isn't problematic with the address create like LISTNAME-{owner|request|bounces|confirm| ..} in address table?

@kienanstewart
Copy link
Contributor Author

kienanstewart commented May 8, 2019 via email

@lelutin
Copy link

lelutin commented Sep 29, 2020

this would be interesting to fix since users usually expect to be able to have more valid characters.

however, the commit that actually added the limitation in 2016, c760e7a, did so in order to avoid shell command injection within the cronjob.

so if we don't want to regress back to that security issue, we should add to this PR a rewrite of the cronjob to a php script so that we avoid all of the nasty pitfalls that come from bash.

@camlafit camlafit deleted the branch AlternC:master November 17, 2023 13:05
@camlafit camlafit closed this Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to create mailing lists with a '-' or '_' in the list address
4 participants