Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

examples/protocols/smtp_client (IDFGH-4240) #6098

Closed
nopnop2002 opened this issue Nov 10, 2020 · 1 comment
Closed

examples/protocols/smtp_client (IDFGH-4240) #6098

nopnop2002 opened this issue Nov 10, 2020 · 1 comment

Comments

@nopnop2002
Copy link

Environment

freeRTOS version:V8.2.0
NEWLIB version:3.0.0
lwIP version:2-1-3-0
ESP-IDF version:v4.3-dev-1197-g8bc19ba89

Problem Description

Incorrect email header.

Incorrect:
There is no line feed code between the header and the body.
Incorrect Contet-Type.

--XYZabcd1234
Content-Type: image/image/png;name=esp_logo.png
Content-Transfer-Encoding: base64
Content-Disposition:attachment;filename="esp_logo.png"
iVBORw0KGgoAAAANSUhEUgAAARgAAAEYCAIAAAAI7H7bAAB4yElEQVR4nOxdBWBU9R9/eZ3rYME2YrSCKAYWBhio2GBgI4piYf7FwMZWsFAMBEVsRVQUW0SlB2OM

Correct:

--XYZabcd1234
Content-Type: image/png;name=esp_logo.png
Content-Transfer-Encoding: base64
Content-Disposition:attachment;filename="esp_logo.png"

iVBORw0KGgoAAAANSUhEUgAAARgAAAEYCAIAAAAI7H7bAAB4yElEQVR4nOxdBWBU9R9/eZ3rYME2YrSCKAYWBhio2GBgI4piYf7FwMZWsFAMBEVsRVQUW0SlB2OM
@github-actions github-actions bot changed the title examples/protocols/smtp_client examples/protocols/smtp_client (IDFGH-4240) Nov 10, 2020
@Alvin1Zhang
Copy link
Collaborator

@nopnop2002 Thanks for reporting and sorry for late reply, we have a fix under internal reviewing and this issue will be closed automatically once the fix is available on GitHub. Thanks.

espressif-bot pushed a commit that referenced this issue Jul 28, 2021
@vikramdattu @espressif-bot
Add newline separator to fix wrong attachment sent
81de14b 
Issue:

Missing newline between header and attachment

Issue seen with email clients viz., AppleMail and YahooMail. Attachment is not renderable.
Some clients like Google mail, Samsung mail tolerate the issue.

Fix:
Added newline between header and attachment.

Closes #6098

Signed-off-by: Vikram Dattu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nopnop2002 @Alvin1Zhang