llhttp 9.0.0 no longer accepts bare LF #236

Activity

1. ShogunPanda commented on Aug 5, 2023

   

   ShogunPanda

   on Aug 5, 2023

   Contributor

   More actions

   I think this is an unwanted regression. I'll take a look soon.
2. Dreamsorcerer commented on Aug 7, 2023

   

   Dreamsorcerer

   on Aug 7, 2023

   Contributor

   More actions

   Which option will need to be enabled after the PR lands to enable this (had a couple of users report similar issues)?
   llhttp_set_lenient_optional_lf_after_cr()?
3. ShogunPanda commented on Aug 9, 2023

   

   ShogunPanda

   on Aug 9, 2023

   Last edited by ShogunPanda

   Contributor

   More actions

   You'll have to use llhttp_set_lenient_headers. The bug was not on the new flags but on old ones :)
4. ShogunPanda

   closed this as completedin #239on Aug 21, 2023