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

plain text alert emails #29

Closed
miekg opened this issue Aug 19, 2019 · 5 comments
Closed

plain text alert emails #29

miekg opened this issue Aug 19, 2019 · 5 comments

Comments

@miekg
Copy link

@miekg miekg commented Aug 19, 2019

The current alert email are html formatted, but somewhat weird, my mutt converts this to

[-- Autoview using /usr/bin/w3m -I 'UTF-8' -dump -T text/html '/home/miek/tmp/mutt.html' --]
A crash was introduced in rewrite

---------
You can unsubscribe from these alerts via the dashboard

And doesn't actually render any links.

anyway it would be nice to have the option to just receive plain text emails.

@miekg miekg changed the title plain text altert emails plain text alert emails Aug 19, 2019
@yevgenypats
Copy link
Collaborator

@yevgenypats yevgenypats commented Aug 19, 2019

I can add this option though then there will be no links (at least not clickable). I would prefer it though just to work.
I made some change to the html (this is the simplest html with single tag), It should work now. can rerun it in a new target and see if the email is now well formatted?

@yevgenypats
Copy link
Collaborator

@yevgenypats yevgenypats commented Sep 11, 2019

@miekg do you know if this was fixed. Do you still receive broken emails?

@miekg
Copy link
Author

@miekg miekg commented Sep 13, 2019

@yevgenypats
Copy link
Collaborator

@yevgenypats yevgenypats commented Sep 13, 2019

ok got it. I'll keep an eye on the go modules support. We can wait a week or so and if it doesn't get in we can use fuzzitdev/go-fuzz fork that support go modules till it will be upstreamed. let me know if this is something you need/want.

closing the issue for now as it seems that the issue was fixed.

@miekg
Copy link
Author

@miekg miekg commented Sep 16, 2019

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.