Skip to content

EMAIL_USE_TLS/EMAIL_USE_SSL are mutually exclusive, so only set one o…#636

Merged
maxking merged 2 commits intomaxking:mainfrom
raph-topo:patch-5
Jun 20, 2023
Merged

EMAIL_USE_TLS/EMAIL_USE_SSL are mutually exclusive, so only set one o…#636
maxking merged 2 commits intomaxking:mainfrom
raph-topo:patch-5

Conversation

@raph-topo
Copy link
Copy Markdown
Contributor

@raph-topo raph-topo commented Jun 19, 2023

Fix #624 like this?

Copy link
Copy Markdown
Owner

@maxking maxking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use backticks for variables.

@maxking
Copy link
Copy Markdown
Owner

maxking commented Jun 20, 2023

Overall, lgtm! Added backticks to the variables.

@maxking maxking enabled auto-merge June 20, 2023 02:54
@maxking maxking merged commit 0745e37 into maxking:main Jun 20, 2023
@raph-topo
Copy link
Copy Markdown
Contributor Author

Thanks! (I copy-pasted from the Django error message as is.)

@raph-topo raph-topo deleted the patch-5 branch June 20, 2023 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] mailman-web: EMAIL_USE_TLS/EMAIL_USE_SSL

2 participants