sozu icon indicating copy to clipboard operation
sozu copied to clipboard

refactoring the HTTP header parser

Open Geal opened this issue 8 years ago • 2 comments
trafficstars

In gitlab by @Geal on Dec 5, 2016, 18:13

The approach using an enum worked quite well to separate the various steps in parsing headers, but now it needs to have states combining length, host, connection, upgrade protocol. Until the end of the headers is reached, the various values we need to gather should be in a structure with optional fields. This will reduce the number of states, and we can still check if something has been already parsed or not.

The new state machine would keep the various end states.

Geal avatar Jan 18 '17 14:01 Geal

@Geal This refactoring is still needed?

Do you have some more details to provide?

Freyskeyd avatar Oct 17 '17 17:10 Freyskeyd

in progress on the dev branch

Geal avatar Jul 29 '21 10:07 Geal