Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: use Base64.strict_encode64 instead of Base64.encode64 #495

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yujideveloper
Copy link

@yujideveloper yujideveloper commented Aug 2, 2023

Fixes

When I used Base64.encode64 to encode attachments, I got the following error:

{
  "errors": [
    {
      "message": "The attachment content must be base64 encoded.",
      "field": "attachments.0.content",
      "help": "http://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/errors.html#message.attachments.content"
    }
  ]
}

It is resolved by changing to encode attachments using Base64.strict_encode64 where no line feeds are added.

Checklist

  • I acknowledge that all my contributions will be made under the project's license
  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I have read the Contribution Guidelines and my PR follows them
  • I have titled the PR appropriately
  • I have updated my branch with the main branch
  • I have added tests that prove my fix is effective or that my feature worksx
  • I have added the necessary documentation about the functionality in the appropriate .md file
  • I have added inline documentation to the code I modified

If you have questions, please file a support ticket, or create a GitHub Issue in this repository.

Attachments must be base64 encoded, using `Base64.strict_encode64` where no
line feeds are added.
@yujideveloper yujideveloper force-pushed the bugfix/fix-base64-encoding-for-attachment branch from 7892bc6 to fd2b502 Compare December 26, 2024 08:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant