Ethsim2
6 سبتمبر 2026، 7:01ص
3
أؤكد وجود صورة إصدار قديمة من تغيير آخر في mail-receiver
فتحت هذا طلب المسح (PR)، والذي تم دمجُه في 11 أغسطس:
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.
يُغيّر هذا الطلب مستقبِل البريد من:
post.set_form_data(email: @mail)
إلى المعامل الأحدث email_encoded الذي يتوقعه Discourse.
بعد دمج الطلب، أعدت بناء mail-receiver في 17 أغسطس، لكن الحاوية المُعاد بناؤها حديثاً لا تزال تحتوي على:
post.set_form_data(email: @mail)
أضفت التفاصيل إلى الطلب المدمج هنا:
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?
أعدت البناء مرة أخرى في 22 أغسطس، وأبلغتني أمر ./launcher rebuild mail-receiver بأن صورة الإصدار المنشورة محدّثة، مع الهاش:
sha256:2a0d45bb92a783c846a821e635413764eeadb3441ef6190085545a4bf054cb1e
لا يزال البريد الوارد يعمل على تثبيتي، لكن كل بريد واصل يستمر في تحفيز تحذير Discourse:
the email parameter is deprecated
لأن الصورة المنشورة لا تحتوي على تغيير #37 المدمج.
لذا يبدو أن هذا يؤكد بشكل مستقل المشكلة الأولى التي وجدتها: التغييرات المدمجة في مستودع mail-receiver discourse/mail-receiver:release المنشورة.
إعجابَين (2)