tsvwg / draft-ietf-tsvwg-udp-options

0 stars 0 forks source link

Could we avoid being judgmental about design of middleboxes? #17

Closed gorryfair closed 9 months ago

gorryfair commented 9 months ago

I personally do dislike the words chosen here : “The design enables traversal of misbehaving middleboxes that incorrectly compute the UDP checksum over the entire IP payload”. To me, it is great for a research paper, but awkward for a standard because it sounds like a judgment on the way others do things, well maybe it is. Could we we can find more positive text and say what it does?

Such as “This design enables traversal of a middlebox that has been designed to compute the UDP checksum over the entire IP payload”. This also applies to later use of "misbehaving middlebox"

jtouch commented 9 months ago

This is a standards document. Middleboxes that compute UDP checksums based on IP lengths are objectively misbehaving and incorrect in their computation. This isn't a research assessment. If it belongs anywhere, it belongs in a spec affected by that decision, IMO. We previously called them misbehaving and errant. They're wrong in what they do. Is there a preferred term for such that DOES NOT imply that they're not flat-out wrong?

I.e., could we NOT avoid being judgmental about protocol standards violations?

gorryfair commented 9 months ago

I'd persoanlly prefer to just say middleboxes that do not comply with RFCxxxx. That's more polite and helpful than saying "errant" or "misbehaving".

jtouch commented 9 months ago

Polite to whom? Middleboxes don't have feelings or practice etiquette. ;-) Although I see your point, we need to use stronger language. Failure to comply is one thing, but these middleboxes create non-interoperability. It's not like they're setting UDP==0 in situations they shouldn't - which is failure to comply, but might not have an impact.

Mike-Heard commented 9 months ago

I thought the word "errant" (which was used before) was more elegant than "misbehaving" 😃

jtouch commented 9 months ago

Errant is fine to me; I can unify around that term if useful.

gorryfair commented 9 months ago

I agree: Errant with respect to the spec seems a statement of fact!

jtouch commented 9 months ago

Changed all occurrences in -24 to "errant"

gorryfair commented 9 months ago

This was agreed at the Interim.