search

martinthomson / http-mice

A progressive integrity content encoding for HTTP
3 stars 2 forks source link

Fix #3: Specify that 0-length payloads are encoded to 0-length messages. #9

Closed jyasskin closed 6 years ago

jyasskin commented 6 years ago

This also includes some incidental fixes around the description of the record size at the beginning of the encoded message.

FYI @davidben and @nyaxt. I went with the 0-len->0-len option because decoders need to deal with content that's too short for the 8-byte record anyway, so I think the special-case fits fairly naturally there. Let me know if that turns out not to be true in the actual implementation.

nyaxt commented 6 years ago

I'd like to defer my vote to @irori.

irori commented 6 years ago

SGTM, this would work for us.

martinthomson commented 6 years ago

All the options are bad. This will do.