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

2.3 Length Requirements #634

Closed
mnot opened this issue Jan 4, 2021 · 7 comments · Fixed by #662
Closed

2.3 Length Requirements #634

mnot opened this issue Jan 4, 2021 · 7 comments · Fixed by #662

Comments

@mnot
Copy link
Member

mnot commented Jan 4, 2021

When a received protocol element is parsed, the recipient MUST be able to parse any value of reasonable length that is applicable to the recipient's role and that matches the grammar defined by the corresponding ABNF rules.

This MUST is very fuzzy, and obviously isn't testable. Should it just be prose?

@mnot mnot added the semantics label Jan 4, 2021
@reschke
Copy link
Contributor

reschke commented Jan 5, 2021

Not convinced, but wouldn't object. Maybe "needs to"?

@royfielding
Copy link
Member

That's what makes the ABNF normative. I don't see a need to change it now.

@mnot
Copy link
Member Author

mnot commented Jan 5, 2021

Sorry, what does the ABNF have to do with this? The issue is the phrase "MUST be able to parse any value of reasonable length", where "reasonable length" is not defined.

@royfielding
Copy link
Member

royfielding commented Jan 7, 2021

It is defined as being reasonable. I have no problem testing that (subjectively). ;-)

@royfielding
Copy link
Member

Sorry, what does the ABNF have to do with this? The issue is the phrase "MUST be able to parse any value of reasonable length", where "reasonable length" is not defined.

I meant the second half.

@royfielding
Copy link
Member

How about

A recipient SHOULD parse a received protocol element defensively, with only marginal expectation that the element conforms to the ABNF and can be parsed within the scope of a reasonable buffer.

@mnot
Copy link
Member Author

mnot commented Jan 7, 2021

I like it

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.

3 participants