That header ...

`If-Match: "ab35ef1bc78", "5be73a9c523"`

Would be interpreted by https://tools.ietf.org/html/rfc7230#section-3.2
and https://tools.ietf.org/html/rfc7230#section-3.2.2

as a Field with a List of 2 values.
Value 1: ab35ef1bc78
Value 2: 5be73a9c523

ETag is not special here, the only listed special case handling for
field-value in the spec is `Set-Cookie`.
See: https://tools.ietf.org/html/rfc7230#section-3.2.2

- Joakim
Joakim Erdfelt / [email protected]


On Thu, Oct 29, 2020 at 9:47 AM Nils Kilden-Pedersen <[email protected]>
wrote:

> Yes, that's my point. ETags *require* quotes, but they're being removed.
>
> On Thu, Oct 29, 2020 at 9:25 AM Tamás Cservenák <[email protected]>
> wrote:
>
>> To me it seems does requite DQUOTE around it...
>> https://tools.ietf.org/html/rfc7232#section-2.3
>>
>> Thanks
>> T
>>
>> On Thu, Oct 29, 2020 at 3:07 PM Nils Kilden-Pedersen <[email protected]>
>> wrote:
>>
>>> After debugging for a weird problem the entire day yesterday, I finally
>>> figured out why my ETag value was losing its quotes. For some reason,
>>> the innocently sounding HttpField.getValues apparently (needlessly?)
>>> appears to assume that comma separated header values might be quoted (not
>>> sure why, I don’t think it’s spec), and thus removes *required* quotes
>>> from etags, causing parsing failure later.
>>>
>>> So, e.g. this header:
>>>
>>> If-Match: "ab35ef1bc78", "5be73a9c523"
>>>
>>> will return invalid ETag values (unquoted).
>>>
>>> I would consider this a bug, but not sure why quote removal is even
>>> there, as I don’t see the HTTP spec requiring quotes around comma separated
>>> values.
>>> _______________________________________________
>>> jetty-users mailing list
>>> [email protected]
>>> To unsubscribe from this list, visit
>>> https://www.eclipse.org/mailman/listinfo/jetty-users
>>>
>> _______________________________________________
>> jetty-users mailing list
>> [email protected]
>> To unsubscribe from this list, visit
>> https://www.eclipse.org/mailman/listinfo/jetty-users
>>
> _______________________________________________
> jetty-users mailing list
> [email protected]
> To unsubscribe from this list, visit
> https://www.eclipse.org/mailman/listinfo/jetty-users
>
_______________________________________________
jetty-users mailing list
[email protected]
To unsubscribe from this list, visit 
https://www.eclipse.org/mailman/listinfo/jetty-users

Reply via email to