Continuing the discussion from HTML email signature not being stripped out of notification reply:
It’s back, although now with extra email quoting inclusion as well. 
Yes this is for @zogstrip
Anyone who wants the original message source via Mess feel free to ping me.
1개의 좋아요
This should now be fixed for good 
https://github.com/discourse/discourse/commit/46b6c55197a41e13c163f10e00138d78d6e79820
5개의 좋아요
Can you send me the raw email?
Of course; check your messages.
1개의 좋아요
Looks like it’s working as expected here. (Except for the fact that we can’t properly identify user signatures and new lines, which are really hard problems)
Is there a preferred format for email signatures that is easily strippable by Discourse? If so we, could let our members know the format.
1개의 좋아요
Looks like this is optimized for stripping out Mobile signatures:
Our biggest offenders come from Desktop clients, for example:
In our case, we could gently encourage our users to proceed their signatures with something like: --------------------<line break> if that is acceptable.
Has this option already been considered: http://blog.mailgun.com/open-sourcing-our-email-signature-parsing-library/ ?
We have re-written our library over the years, it is indeed inspired by MailGun Talon and other libs.
https://github.com/discourse/email_reply_trimmer
I think this is fixed.
1개의 좋아요