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

Correct TO string is not working anymore after upgrade #714

Open
dimm0 opened this issue Jul 12, 2024 · 4 comments
Open

Correct TO string is not working anymore after upgrade #714

dimm0 opened this issue Jul 12, 2024 · 4 comments
Assignees
Labels
bug Something isn't working.

Comments

@dimm0
Copy link

dimm0 commented Jul 12, 2024

Describe the bug

After upgrading to 0.7.1 the correct TO string like To: Name Lastname <name@aaa.com>\r\n is causing the error:

Was expecting RCPT arg syntax of TO:

Just the name@aaa.com works.

Steps to reproduce

Send mail to above recipient

Log files

Nothing in logs, but client is getting the error

  • maddy version: 0.7.1
@dimm0 dimm0 added the bug Something isn't working. label Jul 12, 2024
@foxcpp
Copy link
Owner

foxcpp commented Jul 23, 2024

go-smtp update included in 0.7.1 removed the "lax" parser for RCPT argument, that might be the reason.

I need a log with I/O to troubleshoot the issue further. Add this to your config:

smtp tcp://0.0.0.0:25 {
  io_debug
  debug
  ...
}

and try again. Look for RCPT line in the log and post it here.

@foxcpp foxcpp self-assigned this Jul 23, 2024
@senner
Copy link

senner commented Aug 22, 2024

I get something similar..

submission: incoming message {"msg_id":"e0762daa","sender":"test@sender.net","src_host":"185217ff75c7","src_ip":"4.5.3.3:33184","username":"test@sender.net"}
submission: RCPT ok {"msg_id":"e0762daa","rcpt":"test@example.com"}
submission: DATA error {"addr":"('Test UserFullName', 'test@example.com')","modifier":"submission_prepare","msg_id":"e0762daa","reason":"mail: no angle-addr","smtp_code":554,"smtp_enchcode":"5.6.0","smtp_msg":"Invalid address in To"}
submission: aborted {"msg_id":"e0762daa"}

@foxcpp
Copy link
Owner

foxcpp commented Aug 23, 2024

submission: DATA error {"addr":"('Test UserFullName', '[test@example.com](mailto:test@example.com)')","modifier":"submission_prepare","msg_id":"e0762daa","reason":"mail: no angle-addr","smtp_code":554,"smtp_enchcode":"5.6.0","smtp_msg":"Invalid address in To"}

Looks like you have From: Test UserFullName 'test@example.com' in your message whereas the standard syntax is From: Test UserFullName <test@example.com>. maddy enforces additional checks for messages created via submission.

@senner
Copy link

senner commented Aug 23, 2024

Yah the test is coming from the Authentik server. I'll go investigate there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working.
Projects
None yet
Development

No branches or pull requests

3 participants