Skip to content

Conversation

@s-ichikawa
Copy link
Owner

same as a title

Copy link

@binaryk binaryk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Koozza
Copy link

Koozza commented Feb 21, 2022

Found a small bug. Could you change line 192 of the SendgridTransport class to:

    'type' => $this->getAttachmentContentType($attachment),

And add the following method:

    private function getAttachmentContentType(Datapart $dataPart): string
    {
        return $dataPart->getMediaType() . '/' . $dataPart->getMediaSubtype();
    }

This fixes the attachments not being readable by some mail clients!

@s-ichikawa s-ichikawa merged commit 29aa94a into master Feb 23, 2022
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.

4 participants