Skip to content

Added SES Driver#79

Open
mabadir wants to merge 1 commit into
beyondcode:masterfrom
mabadir:master
Open

Added SES Driver#79
mabadir wants to merge 1 commit into
beyondcode:masterfrom
mabadir:master

Conversation

@mabadir

@mabadir mabadir commented Dec 27, 2020

Copy link
Copy Markdown

Hello,
This PR basically adds SES support to Laravel Mailbox. It is overcoming few shortcomings from Amazon SNS.

  1. The POST request from SNS has content-type as text instead of JSON.
  2. SNS posts a request first without the Authorization header, and expects a response of 401 Unauthorized and Realm.

Appreciate your review and comments.

@denitsa-md

Copy link
Copy Markdown

Question here: am I correct in thinking that this would not work for emails with attachments? As SNS has a limit of 150 kb for the email size?

@ahmetkorkmaz3

Copy link
Copy Markdown

Is it possible to merge this pr? Or do you have any idea for amazon ses?

@dgillier

Copy link
Copy Markdown

Could this PR corrected and merged ?
Thanks :)

@dgillier

dgillier commented Jul 8, 2023

Copy link
Copy Markdown

Could this PR be merged ?
Thanks :)

@faustbrian

Copy link
Copy Markdown

@mabadir is there a fixture for this to make sure the request gets parsed correctly?

@kriiv

kriiv commented Aug 1, 2024

Copy link
Copy Markdown

Any news on this?

@mabadir

mabadir commented Aug 8, 2024

Copy link
Copy Markdown
Author

Hi All,
I am not using laravel-mailbox any more, happy for someone else to take on this PR.

@mechelon

mechelon commented Mar 7, 2025

Copy link
Copy Markdown
Member

Happy to merge this as soon as I get some confirmations of people who test this with SES, since we don't use it.
Also, there's documentation still missing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants