Ethsim2
Setembro 6, 2026, 7:01am
3
Posso confirmar a imagem de release desatualizada a partir de outra alteração em mail-receiver
Abri este PR, que foi mesclado em 11 de agosto:
main ← Ethsim12:fix-email-encoded
merged 01:09PM - 11 Aug 26 UTC
## What does this change?
Updates mail-receiver to send incoming MIME messages … using the
`email_encoded` parameter expected by Discourse's `/admin/email/handle_mail`
endpoint.
The message is encoded with `Base64.strict_encode64` before being added to
the form data.
This removes use of the deprecated raw `email` parameter.
## Why?
Discourse currently logs a deprecation notice for every incoming email sent
by mail-receiver:
> the email parameter is deprecated. all POST requests to this route should
> be sent with a base64 strict encoded email_encoded parameter instead
A freshly rebuilt `discourse/mail-receiver:release` still reproduces the
warning because the current mail-receiver code continues to send `email`.
## Tests
Adds regression coverage which verifies that:
- the deprecated `email` parameter is not sent
- `email_encoded` is sent
- the encoded value represents the original MIME message
Tested locally with Ruby 3.2.11:
- `bundle exec rake test` — 16 examples, 0 failures
- `bundle exec rubocop` — no offenses
- `bundle exec stree check ...` — all files matched expected format
This is related to #13, which proposed the same API migration in 2023.
This PR refreshes that change against current `main` and adds regression
coverage.
Ele altera o mail receiver de:
post.set_form_data(email: @mail)
para o parâmetro mais novo email_encoded esperado pelo Discourse.
Após a fusão do PR, reconstruí o mail-receiver em 17 de agosto, mas o container recém-construído ainda continha:
post.set_form_data(email: @mail)
Adicionei os detalhes ao PR mesclado aqui:
main ← Ethsim12:fix-email-encoded
Thanks again for merging this.
I rebuilt `mail-receiver` again today (17 August… ), but the published
`discourse/mail-receiver:release` image still appears to contain the old
implementation.
The rebuild reports the `release` image as up to date, and checking the
freshly rebuilt running container gives:
```text
# docker exec mail-receiver \
grep -n -E 'base64|set_form_data' \
/usr/local/lib/site_ruby/mail_receiver/discourse_mail_receiver.rb
40: post.set_form_data(email: @mail)
```
A newly received email therefore still produces the `email parameter is
deprecated` warning in Discourse.
Since #37 is merged into `main`, does the
`discourse/mail-receiver:release` Docker image need to be rebuilt/published
separately for this change to reach normal self-hosted installations?
Reconstruí novamente em 22 de agosto e o ./launcher rebuild mail-receiver relatou que a imagem de release publicada estava atualizada, com o digest:
sha256:2a0d45bb92a783c846a821e635413764eeadb3441ef6190085545a4bf054cb1e
O recebimento de e-mails ainda funciona na minha instalação, mas cada e-mail recebido continua a disparar o aviso do Discourse:
the email parameter is deprecated
porque a imagem publicada não contém a alteração mesclada em #37 .
Portanto, isso parece confirmar de forma independente o primeiro problema que você encontrou: as alterações mescladas no repositório mail-receiver discourse/mail-receiver:release publicada.
2 curtidas