Andrew Clayton

Results 552 comments of Andrew Clayton

Thanks for all the testing. The only 3 minutes thing I can immediately see in Unit is the `idle_timeout` [setting](https://unit.nginx.org/configuration/#settings)... > Maximum number of seconds between requests in a keep-alive...

> I'll see about uploading sone files so you can build a image with the issue in. Thanks! > thou is there currently anything we can do to stop this...

OK, thanks for testing, so it certainly seems something related to keep-alive requests, I'm just surprised we haven't seen this before... So in an attempt to reproduce this in the...

Thanks, let me see if I can get it going without docker...

Quick question, where's the actual application that's being run?

OK, thanks, so after a considerable amount of python packages later! I have it running. Anything specific I need to do?

Hi, glad you're enjoying Unit! Right, OK. This was always going to be a possible [eventuality](https://github.com/nginx/unit/issues/916#issuecomment-1842986784). I guess this counts as a _regression_ then... Here;s a PoC patch that stuffs...

> Magical! And here I was getting ready to sacrifice my weekend for this... 😆 Thank you! Heh... > From my experience, the X-Rewrite-Url/X-Rewrite-Uri was more common a lot of...

Here's a _much_ simpler patch! ```diff diff --git ./src/nxt_php_sapi.c ./src/nxt_php_sapi.c index b9d9754a..64a88786 100644 --- ./src/nxt_php_sapi.c +++ ./src/nxt_php_sapi.c @@ -1459,6 +1459,9 @@ nxt_php_register_variables(zval *track_vars_array TSRMLS_DC) track_vars_array TSRMLS_CC); } + nxt_php_set_sptr(req, "UNIT_REWRITTEN_PATH",...

@victorserbu2709 Would the [above](https://github.com/nginx/unit/issues/1615#issuecomment-2891915982) work for you?