New line character in windows is "\r\n", current enabler script only removes "\n" character from csv

aws-samples / amazon-guardduty-multiaccount-scripts

This script automates the process of running the GuardDuty multi-account workflow across a group of accounts that are in your control

Apache License 2.0

130 stars 71 forks source link

New line character in windows is "\r\n", current enabler script only removes "\n" character from csv #22

Closed avallecillac closed 5 years ago

avallecillac commented 5 years ago

When running the script loading a csv file created in Windows with End Of File Sequence CRLF, the script will fail to verify email after creating member, member status gets EmailVerificationFailure because it sends the email with "\r" as trailing character so AWS GuardDuty is unable to verify it. I'll be submitting a pull request with the fix

ryanholland commented 5 years ago

Thanks, I have approved the PR.