sendgrid / sendgrid-python

The Official Twilio SendGrid Python API Library
https://sendgrid.com
MIT License
1.54k stars 714 forks source link

misc: Update attachment.md #1060

Closed swills1 closed 12 months ago

swills1 commented 1 year ago

The way the api key is being passed on line 29 is incorrect and will result in an unauthorized error. The Exception as is will complain that message is not an attribute. Please see my two changes.

Fixes

A short description of what this PR does.

Checklist

If you have questions, please file a support ticket.

swills1 commented 1 year ago

My two changes are absolutely minor and I don't think need to go through every hurdle. I'm sure a maintainer can look, see what I am saying, and add themselves if they deem my changes appropriate. My changes are also to an example usage doc. Not the core library.

copyPasteNinja commented 1 year ago

Hello! Sorry to drop by here, I can't help anywhere else :( I'm having trouble using sendgrid in aws lambda. I tried uploading layer, all python lambda version (.7.8.9). Can I please have hint? I'm really stuck :(

swills1 commented 1 year ago

Seems to work now, but definitely wasn't before. Going to test a bit more to be certain.

copyPasteNinja commented 11 months ago

Update: I did, figure it out. I used this terraform module to add my lambda layers and it worked just fine: https://registry.terraform.io/modules/terraform-aws-modules/lambda/aws/latest