-
-
Notifications
You must be signed in to change notification settings - Fork 661
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
I have an issue with transaction.add_body_filter(ct_match, filter) #2290
Comments
We will need more info. Show us some code and the source of the email in
question.
…On Tue, Nov 7, 2017 at 9:26 AM, jetonk ***@***.***> wrote:
Haraka Haraka.js — Version: 2.8.16
Node v8.6.0
OS Linux h2739470 4.4.0-98-generic #121
<#121>-Ubuntu SMP Tue Oct 10
14:24:03 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
openssl OpenSSL 1.0.2g 1 Mar 2016
I'm trying to edit the body content with transaction.add_body_filter(ct,
enc, buff), it won't work, the received email is empty, and just the
subject is ok.
Even when I just return the buff again it doesn't work at all.
But when I send the email as plain text, it works. Any idea?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#2290>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAobY34k509UwWyLEuyLF5SRdmQxrl-Kks5s0GiGgaJpZM4QU3c4>
.
|
I'm sending email from my office365 account.
|
i can confirm got the same problem, |
I can confirm the same problem - I have an internal plugin that adds a body filter. I have found that PR #2187 that fixed bug #2176 is where the behavior change occurred. The specific commit is dc43bdc. I have confirmed that re-adding the following lines seems to fix the issue but am not 100% sure if this is the correct fix. Lines 104 to 105 in 01c3a80
I will try to get an SMTP conversation capture later tonight for an easier reproduction. |
Same here. It happens only for mails with multipart bodies. Works fine for plain text or plain html emails. Is there any plan to fix this? |
I'm trying to edit the body content with transaction.add_body_filter(/text/(plain|html)/, function (ct, enc, buff), it won't work, the received email is empty, and just the subject is ok.
Even when I just return the buff again it doesn't work at all.
But when I send the email as plain text, it works. Any idea?
The text was updated successfully, but these errors were encountered: