http icon indicating copy to clipboard operation
http copied to clipboard

Added check to discard phantom request cookies

Open Bilge opened this issue 4 years ago • 1 comments

This is an improvement over the original implementation because the previous behaviour was to discard the entire set of request cookies if a phantom cookie was encountered, which is probably never the result we want.

Note this still buckles under a=1; (notice the trailing space). This could be solved by trimming the cookie before the comparison to the empty string, but this is left as an exercise for someone who gives more of a shit than I currently do.

Bilge avatar Jul 29 '21 23:07 Bilge

@kelunik What are we doing with this?

Bilge avatar Sep 05 '21 18:09 Bilge

Whoops, seems like I missed this, twice. Thanks!

kelunik avatar Oct 16 '22 20:10 kelunik