Hi all, The attached patch adds support for log buffering when using variables in the access log file name. The use case is this - we use nginx to receive analytics beacons and write them to the access log. We'd like to have a log file per hour that contains the logs of the specific hour. If we use some external script to perform log rotate, we cannot avoid log lines slipping between adjacent fby erankor - Nginx Development
> -----Original Message----- > From: nginx-devel On Behalf Of Maxim Dounin > Sent: Thursday, July 27, 2017 11:54 AM > To: nginx-devel@nginx.org > Subject: Re: Fixed background requests with asynchronous operations. > > The NGX_AGAIN returned from a content phase handler when you started an AIO operation looks wrong: it means that nginx is already writing a response and wasby erankor - Nginx Development
Hi Roman, I got a GitHub issue opened few days ago, that my module doesn't work following this commit - https://github.com/kaltura/nginx-vod-module/issues/645#issuecomment-317027706 The situation is as follows - 1. My module's content phase handler runs and performs an async read request - a. It calls ngx_file_aio_read which returns NGX_AGAIN b. Increments r->blocked c. Sets r->aio to 1by erankor - Nginx Development
I believe it happens because you changed cf->cmd_type, so when nginx tries to parse the second location, it thinks it’s in a scope of ‘my custom module conf’ and not in a scope of ‘server’, and therefore doesn’t recognize the location directive. Nginx doesn’t automatically parse the contents of the block, you have to handle that yourself in your block handler. Usually this isby erankor - Nginx Development
Hi all, In one of our deployments, we need to set up an nginx server behind a forward proxy. We need to have nginx issue the upstream requests using absolute URLs, e.g.: GET http://domain/uri HTTP/1.1 We tried to use a configuration that looks like: location ~ ^/proxy/(.*) { proxy_pass $scheme://proxy-ip:proxy-port/$scheme://$1$is_args$args; } But the problem is that there's no way to pby erankor - Nginx Development
Ping Eran From: Eran Kornblau Sent: Monday, March 6, 2017 9:53 PM To: nginx-devel@nginx.org Subject: Add support for 'define' directive Hi all, I would like to add support for a 'define' directive to nginx, which will allow text substitutions during conf parsing. At this point I'm thinking about adding support only for simple macros - no parameters, no ifdefs etc. Before I start working on theby erankor - Nginx Development
Hi all, I would like to add support for a 'define' directive to nginx, which will allow text substitutions during conf parsing. At this point I'm thinking about adding support only for simple macros - no parameters, no ifdefs etc. Before I start working on the code, I wanted to check whether this makes sense to you and whether you'll be willing to merge it (assuming it's implemented well, of courby erankor - Nginx Development
> Hello! > > On Thu, Mar 02, 2017 at 01:52:58PM +0000, Eran Kornblau wrote: > > [...] > > > # User Eran Kornblau <erankor@gmail.com> # Date 1488462360 18000 > > # Thu Mar 02 08:46:00 2017 -0500 > > # Node ID ecf63ae67df1d5768be40966a5afadb751d894bc > > # Parent 8b7fd958c59f8280d167fe7dd93f1942bfed5876 > > Added missing static specifierby erankor - Nginx Development
> > -----Original Message----- > From: nginx-devel On Behalf Of Maxim Dounin > Sent: Thursday, March 2, 2017 3:37 PM > To: nginx-devel@nginx.org > Subject: Re: Add missing static specifiers > > Hello! > > On Thu, Mar 02, 2017 at 07:15:25AM +0000, Eran Kornblau wrote: > > [...] > > > > > > Fixed, separated the ints and pointers into two blby erankor - Nginx Development
Thanks, updated patch attached. > Hello! > > On Tue, Feb 28, 2017 at 09:20:37PM +0000, Eran Kornblau wrote: > > [...] > > > > > diff -r 8b7fd958c59f -r 4b4b8f5413a4 src/os/unix/ngx_linux_init.c > > > > --- a/src/os/unix/ngx_linux_init.c Mon Feb 27 22:36:15 2017 +0300 > > > > +++ b/src/os/unix/ngx_linux_init.c Tue Feb 28 11:49:07 2017 -0500by erankor - Nginx Development
Thanks for the review! Updated patch attached, comments inline below > Hello! > > > # HG changeset patch > > # User Eran Kornblau <erankor@gmail.com> # Date 1488300547 18000 > > # Tue Feb 28 11:49:07 2017 -0500 > > # Node ID 4b4b8f5413a4a1679d6ad0aa444e29e3f55b6b2a > > # Parent 8b7fd958c59f8280d167fe7dd93f1942bfed5876 > > add missing staticby erankor - Nginx Development
Hi all, Wrote a small script to find missing 'static's in my module - https://github.com/kaltura/nginx-vod-module/blob/master/test/test_static.py I executed it on nginx core and found a few of those too, see attached patch. The logic for finding these was - 1. An exported symbol (found by running readelf on all object files) 2. Appears only in one source file (I could have checkedby erankor - Nginx Development
If I understand your description correctly, I had the same problem - I wanted to expose an internal error code of my module as an nginx variable, so that I could write it to the access log. But after the redirect to the error page, the context was erased and the handler could not retrieve it. The solution I implemented in my case was that when an error occurs, instead of saving it on the moduleby erankor - Nginx Development
You need to stand on the nginx root folder and execute: auto/configure Eran From: nginx-devel On Behalf Of Yohan Rizk Sent: Tuesday, December 20, 2016 4:39 PM To: nginx-devel@nginx.org Subject: Building from source code Hello! My name is yohan, I am interested in helping out with nginx. I've read the links about building the project from the source code in the mercurial repo and I hby erankor - Nginx Development
Ruslan, thank you very much ! Eran -----Original Message----- From: nginx-devel On Behalf Of Ruslan Ermilov Sent: Thursday, December 8, 2016 8:47 PM To: nginx-devel@nginx.org Subject: Map: the "volatile" parameter. details: http://hg.nginx.org/nginx/rev/ec10ce307dc0 branches: changeset: 6832:ec10ce307dc0 user: Ruslan Ermilov <ru@nginx.com> date: Thu Dec 08 17:51:by erankor - Nginx Development
> > > -----Original Message----- > From: nginx-devel On Behalf Of Ruslan Ermilov > Sent: Wednesday, December 7, 2016 11:20 AM > To: nginx-devel@nginx.org > Subject: Re: Add support for 'nocache' flag to map directive > > On Tue, Dec 06, 2016 at 07:42:36PM +0000, Eran Kornblau wrote: > > > > > > -----Original Message----- > > > From: nginx-by erankor - Nginx Development
> > -----Original Message----- > From: nginx-devel On Behalf Of Ruslan Ermilov > Sent: Tuesday, December 6, 2016 9:13 PM > To: nginx-devel@nginx.org > Subject: Re: Add support for 'nocache' flag to map directive > > There was a similar patch circulating locally circa 2013. > I've updated it today. Please give it a try. > I merged these changes manually (I'm stilby erankor - Nginx Development
As I got no objections... :) patch attached Thank you, Eran _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-develby erankor - Nginx Development
> > From: nginx-devel On Behalf Of Sorin Manole > Sent: Tuesday, November 29, 2016 7:19 AM > To: nginx-devel@nginx.org > Subject: Re: Add support for 'nocache' flag to map directive > > Would it make sense to add nocacheable if input_var or one of the variables in the value options is nocacheable? > Yes, that makes perfect sense to me, but that will be harder to implemeby erankor - Nginx Development
Hi all, I created this post yesterday - https://forum.nginx.org/read.php?10,271197 And thought about submitting a patch for it. Before I do, I wanted to check with you whether this is something you'll be willing to accept (assuming the implementation is correct, according to standards, doesn't break things etc.) In short, the idea is to add an optional flag to the map directive that make the mapby erankor - Nginx Development
Hi, I have a variable whose value I'd like to split and save the result into another variable. For example, if the value is <part1><part2><part3>, I'd like to capture <part1><part3> into a variable. I found that I can do this using the map directive, with regular expressions and named captures. Only problem I have is - the variable that I'm splitting is not cacheaby erankor - Ideas and Feature Requests
Hi, An update on this - we found the problem happens when the number of aio contexts (defaults to 32) is exceeded. When that happens nginx falls back to using regular (synchronous) io, and for some reason this makes the kernel not send completion notification for some pending aio requests. Increasing worker_aio_requests to a larger value (we use 1024) solved the problem for us. IMHO, it wouldby erankor - Nginx Mailing List - English
Hi, Recently while reloading/restarting nginx I've been getting errors such as: 2016/09/02 11:13:44 16480#16480: *1234 open socket #123 left in connection 123 After setting `debug_points abort` and checking the core dump, I found that all requests were blocked on file aio (they had r->blocked and r->aio both set to 1) I then looked at the nginx source and saw this comment: /* * Fby erankor - Nginx Mailing List - English