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

Sendgrid strategy decoding issue with Content-Transfer-Encoding: base64 #10

Closed
jhecking opened this issue Jun 3, 2014 · 0 comments · Fixed by #11
Closed

Sendgrid strategy decoding issue with Content-Transfer-Encoding: base64 #10

jhecking opened this issue Jun 3, 2014 · 0 comments · Fixed by #11

Comments

@jhecking
Copy link

jhecking commented Jun 3, 2014

I've noticed an encoding issue for incoming emails with a Content-Transfer-Encoding: base64 header when handled by the Sendgrid strategy. My assumption is that the message content (text as well as html) is already being decoded by Sendgrid. But Sendgrid still passes on the orig. Content-Transfer-Encoding header and the Sendgrid strategy then passes the header when instantiating the Mail::Message instance. When we then access the mail's text content via mail.text_part.body.decoded the content is garbled because the Mail code applies Base64 decoding again to the content that is already decoded by Sendgrid.

I'm trying to get official confirmation from Sendgrid on the handling of the Content-Transfer-Encoding header here: https://community.sendgrid.com/sendgrid/topics/webhook_api_and_content_transfer_encoding_base64.

jhecking pushed a commit to jhecking/incoming that referenced this issue Jun 3, 2014
jhecking pushed a commit to jhecking/incoming that referenced this issue Jun 3, 2014
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

Successfully merging a pull request may close this issue.

1 participant