[GitHub] [httpd] icing commented on pull request #291: Icing/http1 separation

[GitBox <gi...@apache.org>]

icing commented on pull request #291:
URL: https://github.com/apache/httpd/pull/291#issuecomment-1083205179


   I just made a PR with only the API changes from #291.
   
   https://github.com/icing/httpd/pull/1
   
   Since this will never be merged to apache/httpd, I made
   it against my own fork.
   
   I started a review and commented on the new things added.
   Should you find this useful, I'll make other partial PRs
   for core/http1/http2/proxy changes.
   
   Kind Regards,
   Stefan
   
   > Am 28.03.2022 um 14:16 schrieb ylavic ***@***.***>:
   > 
   > 
   > @ylavic commented on this pull request.
   > 
   > In modules/http1/chunk_filter.c:
   > 
   > > @@ -0,0 +1,762 @@
   > +/* Licensed to the Apache Software Foundation (ASF) under one or more
   > 
   > Likewise for http1/http_filters.c, it seems that all the code could be moved from http/http_filters.c or the few things remaining there don't belng in http1/ ?
   > 
   > —
   > Reply to this email directly, view it on GitHub, or unsubscribe.
   > You are receiving this because you authored the thread.
   > 
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@httpd.apache.org
For additional commands, e-mail: notifications-help@httpd.apache.org