Can't attach csv

[tomwwagstaff-product]

Issue Summary

Several months ago I wrote a script that involved sending a csv file in attachment. I tested this locally with success and uploaded it to a virtual machine to run regularly. Now I find that when I try to run this script locally it throws an error, although it continues to run fine on the virtual machine.

If I omit the attachment it runs fine on both systems.

I have no idea if this relates to issue #704

Also I don't know what the difference is between the sendgrid module and the sendgrid-python module...

Steps to Reproduce

1. Try to run the Python function below.

Technical details:

• sendgrid Version: 5.4.1
• Python Version: 3.6

Virtual machine (where it continues to run):

• sendgrid Version: 5.4.0
• Python Version: 3.6

Error:
BadRequestsError: HTTP Error 400: Bad Request

Code

import sendgrid
from sendgrid.helpers.mail import Email, Content, Mail, Attachment, Personalization

def sendGrid(recipient, subject, body, file = None, filename = "attachment.csv", bcc = 0):
    sg = sendgrid.SendGridAPIClient(apikey=os.environ.get('SENDGRID_API_KEY'))
    from_email = Email("myemail@example.com")
    to_email = Email(recipient)
    content = Content("text/plain", body)
    
    mail = Mail(from_email, subject, to_email, content)
    
    if file:
        data = str.encode(file)
        encoded = base64.b64encode(data).decode()
        
        attachment = Attachment()
        attachment.content = encoded
        attachment.type = "text/csv"
        attachment.filename = filename
        attachment.disposition = "attachment"
        attachment.content_id = "Example Content ID"    
        
        mail.add_attachment(attachment)
        
    if bcc == 1:
        mail.personalizations[0].add_bcc(Email("myemail@example.com"))
    
    try:
        response = sg.client.mail.send.post(request_body=mail.get())
    except urllib.HTTPError as e:
        print(e.read())

[JagadeeshBoyidi]

I'm Facing the same issue <sendgrid.helpers.mail.email.Email object at 0x7fa9a65a7690> is not JSON serializable.

Someone can help me out

[tomwwagstaff-product]

These examples are not the same as the issue I opened here, being a different error thrown in a different use case using a different version of Python. Can you open a separate issue to avoid confusion?

[thinkingserious]

Hello @tomwwagstaff-product,

What is the value of print(response.body) right after you make your post?

Also, can you check if the local version of the SendGrid library is the same version?

@JagadeeshBoyidi,

This issue is not related, could you please open a new issue? I'm guessing you may have this issue.

With Best Regards,

Elmer

[tomwwagstaff-product]

Hello @thinkingserious, thanks for getting back to me

I'm afraid that response never gets successfully assigned because of that error - if I try printing response.body in debug mode it just says NameError: name 'response' is not defined

So the libraries are not quite the same - v5.4.1 vs v5.4.0 - I'm using Anaconda locally and I only have a limited range of versions available. I'm loathe to update the library on the virtual machine as it's the one instance that's still working...

[thinkingserious]

Can you please print the value of mail.get() right before the post?

[tomwwagstaff-product]

Sure, here you go:

{
  "attachments": [
    {
      "content": "YSxiCjEsMgozLDQK",
      "content_id": "Example Content ID",
      "disposition": "attachment",
      "filename": "myFile.csv",
      "type": "text/csv"
    }
  ],
  "content": [
    {
      "type": "text/plain",
      "value": "Hi,\nAttached is a csv file"
    }
  ],
  "from": {
    "email": "myemail@example.com"
  },
  "personalizations": [
    {
      "bcc": [
        {
          "email": "myemail@example.com"
        }
      ],
      "to": [
        {
          "email": "myemail@example.com"
        }
      ]
    }
  ],
  "subject": "New charities"
}

[thinkingserious]

Hello @tomwwagstaff-product,

Your JSON looks correct. I would try the following:

1. Double check the location of myFile.csv
2. Check the .csv file for any non-standard encoding. Perhaps try a simple .csv files with just one or two entries.

With Best Regards,

Elmer

[tomwwagstaff-product]

Thanks for your continuing help with this, @thinkingserious

So actually there is no .csv file saved anywhere, the file that gets passed to that function is just the following string:

a,b
1,2
3,4

Given that, I think it will just be Unicode anyway, right?

Cheers,
Tom

[thinkingserious]

Hi @tomwwagstaff-product,

Can you verify that the base64 decoded file converts to the .csv file?

[tomwwagstaff-product]

Hello @thinkingserious - how can I do that?

[thinkingserious]

You can use this or you can try a web based service.

[tomwwagstaff-product]

Thanks @thinkingserious,

Yeah it decodes back to csv just fine:

base64.b64decode(encoded)
b'a,b\n1,2\n3,4\n'