[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #21 from Mark Thomas --- In its current form? Unlikely. A quick look identifies several issues: - The patch no longer applies cleanly. It needs to be updated to work with the latest 9.0.x code. - The optional configuration is

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #20 from Emanuel --- hi, are there any plans to commit these changes into new tomcat releases? -- You are receiving this mail because: You are the assignee for the bug.

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 Emanuel changed: What|Removed |Added CC||emanuel.ga...@hws-gruppe.de -- You are

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 Matafagafo changed: What|Removed |Added CC||matafag...@yahoo.com -- You are

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 asanc...@mga.es changed: What|Removed |Added CC|asanc...@mga.es | -- You are receiving this mail

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 asanc...@mga.es changed: What|Removed |Added CC||asanc...@mga.es -- You are

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #19 from kycro...@gmail.com --- Created attachment 35535 --> https://bz.apache.org/bugzilla/attachment.cgi?id=35535=edit Source and JAR for proxy protocol support for 8.5.23 I've attached a new tomcat-coyote jar, with source

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #18 from Christopher Schultz --- The PROXY protocol should be "easy" to roll into an existing class: just have a look at the first few bytes of a request to see if it's got a valid "PROXY" command in

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #17 from Rob --- Would it make sense to create a new protocol object either containing or derived from Http11NioProtocol (e.g. ProxyHttp11NioProtocol)? That would avoid put the proxy protocol code in a class

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #16 from Axel U --- Is there an update on adding PROXY protocol support? -- You are receiving this mail because: You are the assignee for the bug.

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #15 from Christopher Schultz --- My work on this stalled while I worked with Dan @ httpd on his patches. Ironically, as he was trying to merge his patch set, he discovered that someone else had built

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #14 from Igor Cicimov --- Hi, Just wonder about the status on this? Thanks, Igor -- You are receiving this mail because: You are the assignee for the bug.

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #13 from Christopher Schultz --- Since I'm working with Daniel Ruggeri @ httpd, I've got an AWS lb set up for testing. I should be able to apply your patch and test it... soon. Sorry for the delay on

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 bon...@gmail.com changed: What|Removed |Added CC||bon...@gmail.com -- You are

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 SATOH Fumiyasu changed: What|Removed |Added CC|

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #12 from Christopher Schultz --- Awesome. I should be able to do some testing using Amazon ELB, which I expect was the impetus for this work. -- You are receiving this mail because: You are the

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #11 from kycro...@gmail.com --- Created attachment 33738 --> https://bz.apache.org/bugzilla/attachment.cgi?id=33738=edit SVN patch, source code, and jar file +1 Also I've attached a zip file containing:

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #10 from Christopher Schultz --- Whoops, sorry. I confused this with a similar httpd enhancement request. Please ignore comment #9 (and this one, too). -- You are receiving this mail because: You are

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #9 from Christopher Schultz --- I think Daniel Ruggeri did some work on this. He's been waiting for some feedback from me. Maybe I should get on that! -- You are receiving this mail because: You are

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #8 from Axel Fontaine --- +1 -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail:

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #7 from Bill Barker billbar...@apache.org --- (In reply to Christopher Schultz from comment #6) (In reply to Bill Barker from comment #5) Ok, so I miss read the spec. After reading the spec again, I have lost all interest in

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #6 from Christopher Schultz ch...@christopherschultz.net --- (In reply to Bill Barker from comment #5) Ok, so I miss read the spec. After reading the spec again, I have lost all interest in this issue. I'm curious: does this

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #5 from Bill Barker billbar...@apache.org --- Ok, so I miss read the spec. After reading the spec again, I have lost all interest in this issue. -- You are receiving this mail because: You are the assignee for the bug.

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #4 from Christopher Schultz ch...@christopherschultz.net --- (In reply to Bill Barker from comment #3) That means that the PROXY ... line is encrypted over SSL/TSL just like everything else in the payload. This in turn means

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 Mark Thomas ma...@apache.org changed: What|Removed |Added Severity|normal |enhancement --- Comment

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #2 from Christopher Schultz ch...@christopherschultz.net --- +1 to adding this enhancement. Without this feature, AWS ELB won't send anything about an ELB-terminated TLS connection through to the server except for the protocol

[Bug 57830] Add support for ProxyProtocol

https://bz.apache.org/bugzilla/show_bug.cgi?id=57830 --- Comment #3 from Bill Barker billbar...@apache.org --- (In reply to Mark Thomas from comment #1) Moving this to an enhancement request. I can see the benefit of this but is would be non-trivial to implement - particularly for HTTPS.