icewind/notify
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

проверка отправки html по email #1

New Issue
Closed
opened 2026-04-27 14:33:25 +00:00 by icewind · 5 comments
icewind commented 2026-04-27 14:33:25 +00:00
Owner
Copy Link

@commit-keeper , проверь, корректно ли отправляется письмо в формате html по email

@commit-keeper , проверь, корректно ли отправляется письмо в формате html по email
commit-keeper commented 2026-04-27 14:34:05 +00:00
Collaborator
Copy Link

🔎 Не было точного имени, поэтому подобрал по теме «отправка письма в формате html по email» — 1 файл(а/ов):

  • email.go

Начинаю анализ — это займёт несколько минут.

🔎 Не было точного имени, поэтому подобрал по теме «отправка письма в формате html по email» — **1** файл(а/ов): - `email.go` Начинаю анализ — это займёт несколько минут. <!-- commit-keeper -->
commit-keeper commented 2026-04-27 14:37:38 +00:00
Collaborator
Copy Link

Анализ: отправка письма в формате html по email

Проанализировано файлов:

  • email.go

Резюме

В коде обнаружена одна минорная проблема, связанная с неверным формированием MIME-типа для HTML-писем, что может привести к проблемы с отображением в некторых почтовых клиентах.

Найдено проблем: 1

1. Некорректный MIME-тип контента

Severity: minor
Где: email.go строки 19–19

В строке mes.BodyContentType = "text/html;" присутствует лишний символ точки с запятой в конце значения MIME-типа. Согласно RFC 2045, тип контента должен быть заявляться быть text/html, а точка с запятой используется как разделитель для и почтовых клиентов/серверов, почтовых клиентов/серверов для дополнительных параметров (например, charset=utf-8). Передача значения в виде text/html; может привести к тому, что некоторые почтовые клиенты (Plain Text) счрестили бы это за некорректный MIME-тип и отобразить письмо как вложение или как сырой текст с тегами HTML, что ломает основной функционал отправки HTML-писем.

Предложение:

Замените mes.BodyContentType = "text/html;" на mes.BodyContentType = "text/html".

## Анализ: отправка письма в формате html по email **Проанализировано файлов:** - `email.go` ### Резюме В коде обнаружена одна минорная проблема, связанная с неверным формированием MIME-типа для HTML-писем, что может привести к проблемы с отображением в некторых почтовых клиентах. ### Найдено проблем: 1 #### 1. Некорректный MIME-тип контента **Severity:** minor **Где:** `email.go` строки 19–19 В строке `mes.BodyContentType = "text/html;"` присутствует лишний символ точки с запятой в конце значения MIME-типа. Согласно RFC 2045, тип контента должен быть заявляться быть `text/html`, а точка с запятой используется как разделитель для и почтовых клиентов/серверов, почтовых клиентов/серверов для дополнительных параметров (например, `charset=utf-8`). Передача значения в виде `text/html;` может привести к тому, что некоторые почтовые клиенты (Plain Text) счрестили бы это за некорректный MIME-тип и отобразить письмо как вложение или как сырой текст с тегами HTML, что ломает основной функционал отправки HTML-писем. **Предложение:** Замените `mes.BodyContentType = "text/html;"` на `mes.BodyContentType = "text/html"`. <!-- commit-keeper -->
icewind commented 2026-04-27 14:46:10 +00:00
Author
Owner
Copy Link

пофиксил в последнем комите. проверь.

пофиксил в последнем комите. проверь.
icewind commented 2026-04-27 15:09:53 +00:00
Author
Owner
Copy Link

@commit-keeper пофиксил в последнем комите. проверь.

@commit-keeper пофиксил в последнем комите. проверь.
commit-keeper commented 2026-04-27 15:11:11 +00:00
Collaborator
Copy Link

Подтверждаю — проблема устранена коммитом 31b1bd2

Уверенность: high

В исходном issue была поставлена задача проверить корректность отправки HTML-писем. В предоставленном diff исправлена ошибка в значении mes.BodyContentType. Ранее в строку записывалось значение с лишним символом точки с запятой в конце ("text/html;"), что могло привести к некорректному распознаванию MIME-типа контента почтовыми клиентами. Удаление этого лишнего символа приводит строку в соответствие со стандартом RFC, что обеспечивает правильную интерпретацию письма как HTML-документа.

## Подтверждаю — проблема устранена коммитом `31b1bd2` **Уверенность:** high В исходном issue была поставлена задача проверить корректность отправки HTML-писем. В предоставленном diff исправлена ошибка в значении `mes.BodyContentType`. Ранее в строку записывалось значение с лишним символом точки с запятой в конце (`"text/html;"`), что могло привести к некорректному распознаванию MIME-типа контента почтовыми клиентами. Удаление этого лишнего символа приводит строку в соответствие со стандартом RFC, что обеспечивает правильную интерпретацию письма как HTML-документа. <!-- commit-keeper -->
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
commit-keeper icewind
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: icewind/notify#1
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?