This is an automated email from the ASF dual-hosted git repository.

raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 6ec91627ec GH-48861: [CI] Fix wrong `smtplib.SMTP.send_message` usage 
(#48876)
6ec91627ec is described below

commit 6ec91627ecee34b53291c915d53e674e32ba3367
Author: Sutou Kouhei <[email protected]>
AuthorDate: Fri Jan 16 17:44:54 2026 +0900

    GH-48861: [CI] Fix wrong `smtplib.SMTP.send_message` usage (#48876)
    
    ### Rationale for this change
    
    https://docs.python.org/3/library/smtplib.html#smtplib.SMTP.sendmail uses 
the third argument as e-mail content but 
https://docs.python.org/3/library/smtplib.html#smtplib.SMTP.send_message uses 
the first argument as e-mail.
    
    ### What changes are included in this PR?
    
    * Pass e-mail as the first argument
    * Remove redundant from and to addresses
      * They are extracted from the given e-mail automatically
    
    ### Are these changes tested?
    
    Yes. I sent a test e-mail manually.
    
    ### Are there any user-facing changes?
    
    No.
    * GitHub Issue: #48861
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Raúl Cumplido <[email protected]>
---
 dev/archery/archery/crossbow/reports.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dev/archery/archery/crossbow/reports.py 
b/dev/archery/archery/crossbow/reports.py
index a2c0487a2b..1c6510ea4f 100644
--- a/dev/archery/archery/crossbow/reports.py
+++ b/dev/archery/archery/crossbow/reports.py
@@ -264,7 +264,7 @@ class ReportUtils:
                 smtp.starttls()
             smtp.login(smtp_user, smtp_password)
             message = report.render()
-            smtp.send_message(smtp_user, report.recipient_email, message)
+            smtp.send_message(message)
 
     @classmethod
     def write_csv(cls, report, add_headers=True):

Reply via email to