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

Generate working DKIM keys by adding -traditional flag and update NGINX instructions to avoid breaking certbot #1989

Merged
merged 2 commits into from
Jan 3, 2024

Conversation

securityguy
Copy link
Contributor

Add -traditional option to openssl genrsa to avoid Python DKIM library (dkimpy) error that prevents email from being sent:

dkim.asn1.ASN1FormatError: Unexpected tag (got 30, expecting 02)

Ref: https://bugs.launchpad.net/dkimpy/+bug/1708917

Add -traditional option to openssl genrsa to avoid Python DKIM library (dkimpy) error that prevents email from being sent:

dkim.asn1.ASN1FormatError: Unexpected tag (got 30, expecting 02)

Ref: https://bugs.launchpad.net/dkimpy/+bug/1708917
Include warning to delete /etc/nginx/sites-enabled/default to avoid a conflict that breaks certbot.
@securityguy
Copy link
Contributor Author

securityguy commented Jan 1, 2024

Update NGINX instructions to avoid certbot issues. The default file created when installing NGINX on debian/ubuntu conflicts with the file created in the instructions. Deleting the default file (which is actually a symlink) resolves the issue.

@securityguy securityguy changed the title Generate working DKIM keys by adding -traditional flag Generate working DKIM keys by adding -traditional flag and update NGINX instructions to avoid breaking certbot Jan 1, 2024
Copy link
Contributor

@nguyenkims nguyenkims left a comment

Choose a reason for hiding this comment

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

Thanks!

@nguyenkims nguyenkims merged commit f3127dc into simple-login:master Jan 3, 2024
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.

2 participants