Repository navigation
feat: gzip mail body when content-encoding is set to gzip - #468
Conversation
|
Hi @SendGridDX team, can someone comment on whether this is the desired behavior/feature of the SDK? I plan to add on tests/docs once get a vibe check |
Hey @Bankq changes looks good to me, you can go ahead add the test and doc changes |
[Mail body compression](https://docs.sendgrid.com/api-reference/mail-send/mail-send#mail-body-compression) is supported on V3 api. This patch added the support that when a client sets the content-encoding header to gzip, send compresses the mail body with gzip.
Thank you @kridai ! I've added the test against it. Local docker test and a prod test against api.sendgrid.com using my own API key both works as expected. Regarding doc, I don't find any current doc mentioning |
[Mail body compression](https://docs.sendgrid.com/api-reference/mail-send/mail-send#mail-body-compression) is supported on V3 api. This patch added the support that when a client sets the content-encoding header to gzip, send compresses the mail body with gzip.
Mail body compression is supported on V3 api. This patch added the support that when a client sets the content-encoding header to gzip, send compresses the mail body with gzip.
Checklist