RE: cache: move open to thread pool

2018-08-08 Thread Eran Kornblau
Hi, > > - The code bypasses open file cache, and uses a direct call > in the http cache code instead. While it might be ok in your > setup, it looks like an incomplete solution from the generic point > of view. A better solution would be to introduce a generic > interface in

Re: post body

2018-08-08 Thread Dk Jack
Thank you. Will looking it and get back. Since I need to inspect, I'll follow your suggestion and see if I can use the body filter option. regards, Dk On Wed, Aug 8, 2018 at 11:34 AM Maxim Dounin wrote: > Hello! > > On Tue, Aug 07, 2018 at 10:46:08PM -0700, Dk Jack wrote: > > > Haven't

Re: post body

2018-08-08 Thread Maxim Dounin
Hello! On Tue, Aug 07, 2018 at 10:46:08PM -0700, Dk Jack wrote: > Haven't received any response to inquiry. Would appreciate it if someone > can comment. > > > In my module, I'd like to look into the post body to decide if the request > > should be allowed to proceed to origin server or not.

Re: cache: move open to thread pool

2018-08-08 Thread Maxim Dounin
Hello! On Tue, Aug 07, 2018 at 02:26:01PM -0700, Ka-Hing Cheung via nginx-devel wrote: > commit 0bcfefa040abdff674047c15701d237ff16a5f2b > Author: Ka-Hing Cheung > Date: Fri Aug 3 13:37:58 2018 -0700 > > move open to thread pool > > At cloudflare we found that open() can block a