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

removal of obs-fold from field-value causes inconsistencies #923

Closed
reschke opened this issue Aug 2, 2021 · 1 comment · Fixed by #962
Closed

removal of obs-fold from field-value causes inconsistencies #923

reschke opened this issue Aug 2, 2021 · 1 comment · Fixed by #962

Comments

@reschke
Copy link
Contributor

reschke commented Aug 2, 2021

Citing Ben Kaduk (#915):

A sender MUST NOT generate a message that includes line folding
(i.e., that has any field line value that contains a match to the
obs-fold rule) unless [...]

Since we don't include the obs-fold production as a component of any
other production, and field-value excludes CRLF, it seems that any such
field line value would already be in violation of the ABNF and thus
forbidden. I don't really want to advocate for including obs-fold in
the field-value production in -semantics, though, so maybe accepting
this nit is the least bad choice here.

In RFC723x, field-value allowed obs-fold. We need to:

  • tune the prose above, and
  • somehow properly allow it in message/http (maybe by modifying the field-value for that specific case???)
@reschke
Copy link
Contributor Author

reschke commented Aug 16, 2021

see also #116

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants