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

ABNF recipient list production #element does not allow null element prior to first element #164

Closed
reschke opened this issue Oct 18, 2018 · 9 comments

Comments

@reschke
Copy link
Contributor

reschke commented Oct 18, 2018

See https://www.rfc-editor.org/errata/eid5257.

Mailing list discussion starts here: https://lists.w3.org/Archives/Public/ietf-http-wg/2018JanMar/0131.html

@reschke
Copy link
Contributor Author

reschke commented Mar 26, 2019

See also #14

@reschke
Copy link
Contributor Author

reschke commented Mar 26, 2019

A simple fix for this would be to simplify the list expansion to just map \#element, and to put all other constraints in prose (the current approach doesn't work anyway due to the rules for empty list elements which are not counted).

@reschke
Copy link
Contributor Author

reschke commented Mar 26, 2019

Proposal for recipient list rule mapping:

#element => *( OWS "," ) *( [ OWS element ] OWS "," )

More eyes requested :-) @royfielding @mnot

@reschke
Copy link
Contributor Author

reschke commented Mar 27, 2019

The current proposal is incorrect as it does not allow a singleton element.

@reschke
Copy link
Contributor Author

reschke commented Mar 27, 2019

Maybe:

#element => *( "," OWS ) [ element ] *( OWS "," [ OWS element ] ) *( OWS "," )

better...

#element => [ element ] *( OWS "," [ OWS element ] )

for readability:

#element => [ element ] *( OWS "," OWS [ element ] )

@reschke
Copy link
Contributor Author

reschke commented Mar 27, 2019

PR updated

@royfielding
Copy link
Member

And maybe

1#element => element *( OWS "," OWS [ element ] )

right?

@annevk
Copy link
Contributor

annevk commented Mar 27, 2019

@royfielding that'd outlaw Multiple-Element-Header: , somevalue, which seems wrongish?

@royfielding royfielding changed the title list production ABNF potentially incorrect ABNF recipient list production #element does not allow null element prior to first element Mar 28, 2019
@royfielding
Copy link
Member

Closed by commit e475f36 for the recipient case. Note that this may become obsolete if we switch to only providing ABNF for what is good to send.

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

No branches or pull requests

3 participants