search

aws-solutions / aws-centralized-logging

Apache License 2.0
253 stars 104 forks source link

Can't use email with '+' addressing for AdminEmail parameter #27

Closed milbrya closed 2 years ago

milbrya commented 3 years ago

Describe the bug

If I use and e-mail address with xxxx+yyyy@zzz.com, it template fails to launch with the error message: Parameter 'AdminEmail' must match pattern ^[\w]+\@[\w]+.[a-z]+$

To Reproduce

Enter an e-mail address that has the plus addressing scheme

Expected behavior

I would expect '+' addressing in the e-mail address for AdminEmail.

Please complete the following information about the solution:

(SO0009) - The AWS CloudFormation template for deployment of the centralized-logging. Version v4.0.0",

gsingh04 commented 3 years ago

Thanks for reporting this bug.

We have added it to our backlog items to be shipped with next release.

alb777 commented 3 years ago

Any solution or update regarding this issue ? Iam facing the same problem and cannot deploy cloudformation.

alkayahan commented 3 years ago

@alb777 until my PR is reviewed, you can try from my fork where I fixed this problem https://github.com/alkayahan/aws-centralized-logging/tree/bugfix-emailregex

gsingh04 commented 2 years ago

This has been addressed with release v4.0.1