dart-lang / http_parser

A platform-independent Dart package for parsing and serializing HTTP formats.
https://pub.dev/packages/http_parser
BSD 3-Clause "New" or "Revised" License
38 stars 28 forks source link

fix reading media-type with trailing semicolon #63

Closed kascote closed 1 year ago

kascote commented 1 year ago

Fixes #62

kevmoo commented 1 year ago

Can you point to the RFC, etc that says this is okay? We should also update the changelog...

kascote commented 1 year ago

rfc2616 and rfc9110 both define parameters like

transfer-extension      = token *( ";" parameter )

so I think the current implementation is ok. close this PR and the issue?

kevmoo commented 1 year ago

We should try to follow specs. If the spec says this is illegal, better to close out I think.