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.

Sign up for GitHub

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

Jump to bottom

dkim: add SignOptions.BodyLength #3

Closed
emersion opened this issue Feb 6, 2017 · 2 comments
Closed

dkim: add SignOptions.BodyLength #3

emersion opened this issue Feb 6, 2017 · 2 comments
Labels
enhancement question

Comments

@emersion
Copy link
Owner

emersion commented Feb 6, 2017

As discussed in https://tools.ietf.org/html/rfc6376#section-8.2, this should be used with care.

Maybe we should just choose not to support this, given the security issues it implies.
@emersion emersion transferred this issue from emersion/go-dkim Apr 14, 2019
@emersion emersion changed the title Add SignOptions.BodyLength dkim: add SignOptions.BodyLength Aug 31, 2019
@foxcpp
Copy link
Contributor

foxcpp commented Oct 27, 2019

I vote for not supporting it. maddy considers signatures with body length limit invalid by default.

@emersion
Copy link
Owner Author

Yeah, I agree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@emersion @foxcpp