Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread Eric Covener
I'm testing your recent config and the config from https://github.com/apache/httpd/commit/cab0bfbb2645bb8f689535e5e2834e2dbc23f5a5#commitcomment-20393588 w/ htaccess that makes index.php act like a front controller. Lately I look at the output of the fakefpm.pl script I posted above, but was

Re: Reset out x.minor.z definition of 'minor' at httpd?

2017-01-25 Thread Reindl Harald
Am 26.01.2017 um 00:20 schrieb David Zuelke: On 20.01.2017, at 21:37, Graham Leggett wrote: On 20 Jan 2017, at 7:47 PM, David Zuelke wrote: I'd actually like to question the whole practice of porting features back to older branches. I think that's the

Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread Eric Covener
On Wed, Jan 25, 2017 at 6:12 PM, David Zuelke wrote: >> AddType application/x-php7-fpm .php >> Action application/x-php7-fpm /php7-fpm virtual >> >> SetHandler proxy:fcgi://localhost:9000 >> >> >> setup. This shows what the httpd conf looks like... >> can you

Re: svn commit: r1780328 - in /httpd/httpd/trunk/modules: mappers/mod_actions.c proxy/mod_proxy_fcgi.c

2017-01-25 Thread Eric Covener
On Wed, Jan 25, 2017 at 11:34 PM, Jim Jagielski wrote: > This was tested with: > > AddType application/x-php7-fpm .php > Action application/x-php7-fpm /php7-fpm virtual > > SetHandler proxy:fcgi://localhost:9000 > > > What setup are you looking at? Can you provide

Re: svn commit: r1780328 - in /httpd/httpd/trunk/modules: mappers/mod_actions.c proxy/mod_proxy_fcgi.c

2017-01-25 Thread Jim Jagielski
This was tested with: AddType application/x-php7-fpm .php Action application/x-php7-fpm /php7-fpm virtual SetHandler proxy:fcgi://localhost:9000 What setup are you looking at? Can you provide your config. I'm testing to see if we even need the dconf->backend_type anymore... That

Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread Eric Covener
Seems like w/ the update it's still a bogus PATH_INFO that fpm is correcting 'PATH_INFO' => '/foo.php/bar/baz/', I'm using this as the target from one of the old bug reports. #!/usr/bin/perl use FCGI; use Socket; use FCGI::ProcManager; use Data::Dumper; $num_args = $#ARGV + 1; if

Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread Jim Jagielski
OK... I just committed something that in the AddType/Action setup has reasonable values w/ php-fpm70

Re: svn commit: r1780328 - in /httpd/httpd/trunk/modules: mappers/mod_actions.c proxy/mod_proxy_fcgi.c

2017-01-25 Thread Eric Covener
What's the config for this? I am struggling to get PATH_INFO set in a non SetHandler situation. On Wed, Jan 25, 2017 at 10:52 PM, wrote: > Author: jim > Date: Thu Jan 26 03:52:34 2017 > New Revision: 1780328 > > URL: http://svn.apache.org/viewvc?rev=1780328=rev > Log: > Adjust

Re: AW: svn commit: r1776575 - in /httpd/httpd/trunk: docs/log-message-tags/next-number docs/manual/mod/mod_remoteip.xml modules/metadata/mod_remoteip.c

2017-01-25 Thread Daniel Ruggeri
On 1/25/2017 6:53 PM, Daniel Ruggeri wrote: > I'd say that not returning until ctx->bb has enough information to > determine if the header is present or not would be sufficient. Isn't > this already done in the potentially repeated calls to ap_get_brigade on > line no 1056 inside the loop

Re: AW: svn commit: r1776575 - in /httpd/httpd/trunk: docs/log-message-tags/next-number docs/manual/mod/mod_remoteip.xml modules/metadata/mod_remoteip.c

2017-01-25 Thread Daniel Ruggeri
First, my apologies, but it looks like line wrapping is going to exceed the usual number of columns so formatting may get wonky in this reply. On 1/17/2017 3:48 AM, Plüm, Rüdiger, Vodafone Group wrote: > >> -Ursprüngliche Nachricht- >> Von: Daniel Ruggeri [mailto:drugg...@primary.net] >>

Re: svn commit: r1776575 - in /httpd/httpd/trunk: docs/log-message-tags/next-number docs/manual/mod/mod_remoteip.xml modules/metadata/mod_remoteip.c

2017-01-25 Thread Daniel Ruggeri
To clarify, the issues with handling of the buckets and the fact that buckets are not being set aside came from the original code. The question about what to do regarding f->next versus passing the data along some other way is the only one that came from the code I added to make the header

Re: Reset out x.minor.z definition of 'minor' at httpd?

2017-01-25 Thread David Zuelke
On 20.01.2017, at 21:37, Graham Leggett wrote: > > On 20 Jan 2017, at 7:47 PM, David Zuelke wrote: > >> I'd actually like to question the whole practice of porting features back to >> older branches. I think that's the core reason why trunk is in total >>

Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread David Zuelke
On 25.01.2017, at 20:12, Jim Jagielski wrote: > > Can you provide to me a pgp-fpm.conf that you use... Basically, > I want to create an environ that exactly uses the > > AddType application/x-php7-fpm .php > Action application/x-php7-fpm /php7-fpm virtual > >

Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread Jacob Champion
On 01/25/2017 12:21 PM, Jim Jagielski wrote: OK, got it working and just saw that in these cases, proxy_fcgi_canon() isn't even called... which throws that whole idea out of the water. I argue that this is a bug. It's made other PHP fixes difficult. See [1], about halfway through the message.

Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread Jim Jagielski
OK, got it working and just saw that in these cases, proxy_fcgi_canon() isn't even called... which throws that whole idea out of the water.

Re: Summary: Broken FastCGI with httpd

2017-01-25 Thread Jim Jagielski
Can you provide to me a pgp-fpm.conf that you use... Basically, I want to create an environ that exactly uses the AddType application/x-php7-fpm .php Action application/x-php7-fpm /php7-fpm virtual SetHandler proxy:fcgi://localhost:9000 setup. This shows what the httpd conf

Re: mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Stefan Priebe - Profihost AG
Hi, Am 25.01.2017 um 15:59 schrieb Eric Covener: > On Wed, Jan 25, 2017 at 9:55 AM, Stefan Priebe - Profihost AG > wrote: >> I also saw some more entries with 0 byte size in logs if there is no >> body. http/1.1 logs the header size as well. > > What's your logformat?

Re: mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Eric Covener
On Wed, Jan 25, 2017 at 9:55 AM, Stefan Priebe - Profihost AG wrote: > I also saw some more entries with 0 byte size in logs if there is no > body. http/1.1 logs the header size as well. What's your logformat? Apache doesn't count headers in %b. If it's zero bytes, a -

Re: mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Stefan Priebe - Profihost AG
Am 25.01.2017 um 15:50 schrieb Stefan Eissing: > >> Am 25.01.2017 um 15:31 schrieb Stefan Priebe - Profihost AG >> : >> >> Chrome docs says: >> >> "cancelled / net::ERR_ABORTED is intended to only be generated when a >> user action causes a load to be interrupted. This

Re: mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Stefan Eissing
> Am 25.01.2017 um 15:31 schrieb Stefan Priebe - Profihost AG > : > > Chrome docs says: > > "cancelled / net::ERR_ABORTED is intended to only be generated when a > user action causes a load to be interrupted. This can happen when a new > navigation interrupts an existing

Re: mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Stefan Priebe - Profihost AG
Chrome docs says: "cancelled / net::ERR_ABORTED is intended to only be generated when a user action causes a load to be interrupted. This can happen when a new navigation interrupts an existing one, or when the user clicks the STOP button" This matches my click assumption. I think it's just

Re: mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Stefan Priebe - Profihost AG
Am 25.01.2017 um 14:57 schrieb Yann Ylavic: > Hi, > > On Wed, Jan 25, 2017 at 2:46 PM, Stefan Priebe - Profihost AG > wrote: >> >> Is this a bug or a feature? Is it correct that it logs a 500 code? > > ErrorLog's file should tell more about the error. No error log does

Re: ACC field in server-status

2017-01-25 Thread Eric Covener
On Tue, Jan 24, 2017 at 6:38 PM, Eric Covener wrote: > I have been staring at some different /server-status output lately and > notice that the last two fields in ACC are always the same, even > though one is supposed to be connections by slot and the other > connections by

Re: mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Yann Ylavic
Hi, On Wed, Jan 25, 2017 at 2:46 PM, Stefan Priebe - Profihost AG wrote: > > Is this a bug or a feature? Is it correct that it logs a 500 code? ErrorLog's file should tell more about the error. Regards, Yann.

mod_http2 and canceled requests logs status 500 internal server error code

2017-01-25 Thread Stefan Priebe - Profihost AG
Hello, while digging through some apache logfiles i saw some strange entries: "GET URL HTTP/2.0" 500 0 They all had a 500 status error code and a size of 0. I'm able to reproduce them while clicking fast on a php site. For each log line with status code 500 i get a STATUS (canceled) in my

Fwd: [Bug 53555] Scoreboard full error with event/ssl

2017-01-25 Thread Luca Toscano
Awesome feedback, great work on mpm-event Stefan! -- Forwarded message -- From: Date: 2017-01-25 13:00 GMT+01:00 Subject: [Bug 53555] Scoreboard full error with event/ssl https://bz.apache.org/bugzilla/show_bug.cgi?id=53555 --- Comment #67 Hi Stefan,

Re: mod_http2 and Frequent wake-ups for mpm_event

2017-01-25 Thread Stefan Priebe - Profihost AG
Am 25.01.2017 um 10:46 schrieb Stefan Eissing: > Thanks Yann! > > Stefan: here is the patch as committed to trunk: Up and running. Will report back. > > > Cheers, Stefan > >> Am 25.01.2017 um 01:41 schrieb Yann Ylavic : >> >> Hi Stefan, >> >> On Tue, Jan 24, 2017 at

Re: mod_http2 and Frequent wake-ups for mpm_event

2017-01-25 Thread Stefan Eissing
Thanks Yann! Stefan: here is the patch as committed to trunk: h2_beams_cleanup_v4.diff Description: Binary data Cheers, Stefan > Am 25.01.2017 um 01:41 schrieb Yann Ylavic : > > Hi Stefan, > > On Tue, Jan 24, 2017 at 1:37 PM, Stefan Eissing >