jason/xmpp-email-bridge
Watch
1
0
Fork
You've already forked xmpp-email-bridge
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix: catch SMTPException instead of requiring AUTH extension #6

Merged
jason merged 1 commit from jch/fix-smtp-auth-localhost into main 2026-07-07 21:40:32 -04:00
Conversation 0 Commits 1 Files changed 1 +5 -8
jason commented 2026-07-07 21:38:21 -04:00
Owner
Copy link

The previous commit used server.has_ext('AUTH') which doesn't exist on smtplib.SMTP objects, causing an AttributeError.

This switches the exception from catching only SMTPAuthenticationError to catching all SMTPException subclasses, allowing auth failure (including when AUTH isn't advertised) to gracefully skip and still send via relay.

The previous commit used `server.has_ext('AUTH')` which doesn't exist on `smtplib.SMTP` objects, causing an `AttributeError`. This switches the exception from catching only `SMTPAuthenticationError` to catching all `SMTPException` subclasses, allowing auth failure (including when AUTH isn't advertised) to gracefully skip and still send via relay.
jason merged commit 6a8db15801 into main 2026-07-07 21:40:32 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
jason/xmpp-email-bridge!6
No description provided.