Sending attachments with UTF-8 Characters

[chrisgehg]

I've been trying to send attachments with UTF-8 Characters using the Java api method, email.addAttachment() method, with a file named as 意_hello_字.jpg

Looking like this,
email.addAttachment("意_hello_字.jpg", pathToFile);

After being delivered through email, the image comes through fine, but it is named as ?hello?.jpg . Is this an api error? I found a similar error in the C# api, that I think was fixed by the team, sendgrid/sendgrid-csharp#54

I double checked to make sure that the String was correct in the IDE before it was passed in, and it was.

On top of this, the other email fields, subject and text body work fine with UTF-8 characters using the API.

[chrisgehg]

Fixed it,

In the Sendgrid class I added two lines..

public HttpEntity buildBody(Email email) {
MultipartEntityBuilder builder = MultipartEntityBuilder.create(); //There already

    //Allow for UTF-8 Filenames
    builder.setCharset(Charset.forName("UTF-8"));
builder.setMode(HttpMultipartMode.BROWSER_COMPATIBLE);

This fixed it for me.

[thinkingserious]

@chrisgehg Awesome, thanks for letting us know the fix!

[laurentperezjs]

Hi
Sorry to bump into this, but was this fixed in 2.2.2 ?
I'm running into the same issue, but I can't really make my own jar with the UTF fix, and there's no getter to get to the builder from SendGrid class.
Thanks
laurent

[thinkingserious]

Hi @laurentperezjs,

This has not been fixed in the official library yet, but it's on our roadmap.

In the mean time, could you subclass and override that function?

Thanks!

[thinkingserious]

Hello @laurentperezjs,

We will look into fixing this in the new version of our library: https://github.com/sendgrid/sendgrid-java/tree/v3beta which comes out of beta in a few weeks.

With Best Regards,

Elmer

[thinkingserious]

This problem should be fixed with this merged pull request: sendgrid/java-http-client#5

I'm now updating the dependency on this library.

[jplaza]

I have the exact same issue with the Python lib. Should I open an issue there or is this something that has been fixed?

[thinkingserious]

Hello @jplaza,

Please do open a ticket at the Python lib with the steps it would take to reproduce. Thank you!