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

Unsafe content in HTML emails #59

Open
dcRUSTy opened this issue Jul 12, 2020 · 3 comments
Open

Unsafe content in HTML emails #59

dcRUSTy opened this issue Jul 12, 2020 · 3 comments

Comments

@dcRUSTy
Copy link

dcRUSTy commented Jul 12, 2020

Describe the bug
JavaScript and external resources are allowed in "HTML version" of email which might lead to tracking and IP address leakage, and other wide range of security issues.

To Reproduce
Steps to reproduce the behavior:

  1. Send urself an HTML with inline javascript." <script>alert(1)</script>
  2. In bot click on "View HTML"

Expected behavior
By default they should be blocked OR at least user should be given control.

Screenshots
JSinHTMLEmails

@rosehgal
Copy link
Owner

Hi @dcRUSTy you are right here, I never bothered to take this into account as i thought the unique html link will be only available to the user and there are not user to user mail sending as one user would not even get the email id of any other user and this is done via bruteforce.
But yes this will actually compromise the user IP address and may execute other client side scripts which is definitely an issue. I would like to thank you again for pointing this out. Apart from that there are no session related information leaks with it as there is no sessions involved.

Gimme sometime to see what is the best way would to fix/remediate this out. I will keep this issue open till then.

@dcRUSTy
Copy link
Author

dcRUSTy commented Jul 12, 2020

Thinking out loud.. In addition to above. How about sending the HTML file as attachment in the bot, this way user has a copy of his email irrespective of service works in future or not, also the solves overhead of storing HTML file on server.

@rosehgal
Copy link
Owner

But this wont fix the issue. JS file execution will still be there. Something else is needed.

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

No branches or pull requests

2 participants