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.

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

SmtpResponseParserException: EHLO keyword contains invalid character #8715

Open
2 tasks done
KopiasCsaba opened this issue Jan 3, 2025 · 1 comment
Open
2 tasks done
Labels
status: needs information Needs more information to proceed type: bug Something is causing incorrect behavior or errors unconfirmed Newly reported issues awaiting triage or confirmation

Comments

@KopiasCsaba
Copy link

Checklist

  • I have used the search function to see if someone else has already submitted the same bug report.
  • I will describe the problem with as much detail as possible.

App

Thunderbird for Android

App version

8.2

Where did you get the app from?

F-Droid

Android version

15 - grapheneos

Device model

Pixel 7a

Steps to reproduce

Just send an email

Expected behavior

No exception

Actual behavior

I'm getting an exception.
But the operation seems to be successful.

Logs

I'm getting this:

01-04 00:27:23.659 30196 30222 V SmtpTransport: SMTP <<< 220 mail2.*******.net ESMTP Postcow
01-04 00:27:23.660 30196 30222 D SmtpTransport: SMTP >>> EHLO [127.0.0.1]
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250-mail2.*******.net
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250-PIPELINING
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250-SIZE 104857600
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250-ETRN
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250-STARTTLS
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250-ENHANCEDSTATUSCODES
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250-8BITMIME
01-04 00:27:23.917 30196 30222 V SmtpTransport: SMTP <<< 250 DSN
01-04 00:27:23.921 30196 30222 D SmtpTransport: SMTP >>> STARTTLS
01-04 00:27:24.182 30196 30222 V SmtpTransport: SMTP <<< 220 2.0.0 Ready to start TLS
01-04 00:27:24.193 30196 30222 D SmtpTransport: SMTP >>> EHLO [127.0.0.1]
01-04 00:27:24.231  1258  1258 D AOC     : A3:MSG: usf_sensor.cc, 1713: USF: Received stop sampling request for LSM6DSV Accelerometer (active:3).
01-04 00:27:24.237   983   999 I CHRE    : @ 132366.075: [AR] ON => IDLE
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger: Ignoring EHLO keyword line: AUTH=PLAIN LOGIN
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger: com.fsck.k9.mail.transport.smtp.SmtpResponseParserException: EHLO keyword contains invalid character
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.mail.transport.smtp.SmtpResponseParser.parserError(SmtpResponseParser.kt:414)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.mail.transport.smtp.SmtpResponseParser.checkAndNormalizeEhloKeyword(SmtpResponseParser.kt:138)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.mail.transport.smtp.SmtpResponseParser.parseEhloLine(SmtpResponseParser.kt:122)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.mail.transport.smtp.SmtpResponseParser.readHelloResponse(SmtpResponseParser.kt:108)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.mail.transport.smtp.SmtpTransport.sendHello(SmtpTransport.kt:303)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.mail.transport.smtp.SmtpTransport.open$smtp(SmtpTransport.kt:126)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.mail.transport.smtp.SmtpTransport.sendMessage(SmtpTransport.kt:346)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.backend.imap.ImapBackend.sendMessage(ImapBackend.kt:146)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.controller.MessagingController.sendPendingMessagesSynchronous(MessagingController.java:1544)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.controller.MessagingController$5.run(MessagingController.java:1429)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.controller.MessagingController.runInBackground(MessagingController.java:221)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.controller.MessagingController.-$$Nest$mrunInBackground(MessagingController.java:0)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at com.fsck.k9.controller.MessagingController$1.run(MessagingController.java:162)
01-04 00:27:25.130 30196 30222 V SmtpTransport$logger:  at java.lang.Thread.run(Thread.java:1117)
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-mail2.*******.net
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-PIPELINING
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-SIZE 104857600
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-ETRN
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-AUTH PLAIN LOGIN
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-AUTH=PLAIN LOGIN
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-ENHANCEDSTATUSCODES
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250-8BITMIME
01-04 00:27:25.132 30196 30222 V SmtpTransport: SMTP <<< 250 DSN
01-04 00:27:25.134 30196 30222 D SmtpTransport: SMTP >>> *sensitive*
01-04 00:27:25.442 30196 30222 V SmtpTransport: SMTP <<< 235 2.7.0 [omitted]
01-04 00:27:25.452 30196 30222 D SmtpTransport: SMTP >>> MAIL FROM:<*******@*******.net> BODY=8BITMIME
01-04 00:27:25.456 30196 30222 D SmtpTransport: SMTP >>> RCPT TO:<*******@gmail.com>
01-04 00:27:25.711 30196 30222 V SmtpTransport: SMTP <<< 250 2.1.0 Ok
01-04 00:27:26.573 30196 30222 V SmtpTransport: SMTP <<< 250 2.1.5 Ok
01-04 00:27:26.574 30196 30222 D SmtpTransport: SMTP >>> DATA
01-04 00:27:26.815 30196 30222 V SmtpTransport: SMTP <<< 354 End data with <CR><LF>.<CR><LF>
01-04 00:27:26.830 30196 30222 D SmtpTransport: SMTP >>> .
01-04 00:27:27.638 30196 30222 V SmtpTransport: SMTP <<< 250 2.0.0 Ok: queued as 7216A10008D
01-04 00:27:27.640 30196 30222 D SmtpTransport: SMTP >>> QUIT
@KopiasCsaba KopiasCsaba added type: bug Something is causing incorrect behavior or errors unconfirmed Newly reported issues awaiting triage or confirmation labels Jan 3, 2025
@cketti
Copy link
Member

cketti commented Jan 3, 2025

What do you mean by "I'm getting an exception."? Do you see an error reported outside of the debug log?

The log message itself is correct. EHLO responses may not contain the = character. However, your mail server seems to be Postfix configured with broken_sasl_auth_clients = yes. This adds a non-standard AUTH line to the EHLO response (presumably to support broken clients), which in turn triggers the log message in our code because a non-standard EHLO response line is read.
However, aside from generating the log message, the app will ignore this line. This shouldn't impact the operation of the app.

@cketti cketti added the status: needs information Needs more information to proceed label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs information Needs more information to proceed type: bug Something is causing incorrect behavior or errors unconfirmed Newly reported issues awaiting triage or confirmation
Projects
None yet
Development

No branches or pull requests

2 participants