[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-03-14 Thread persiaAziz
Github user persiaAziz commented on the issue: https://github.com/apache/trafficserver/pull/1503 7.1.x has good amount of works on client certs, so yes this should go in 7.1.x too :) . Like I said, it is unnecessary to check for client cert in the remap function when the cert is only

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1503 What's the word on this PR? 1) Should we land it? 2) Do we still want this to 7.1.x? Note sure why it's been sitting for so long. --- If your project is set up for it, you can reply to this

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-03-13 Thread shinrich
Github user shinrich commented on the issue: https://github.com/apache/trafficserver/pull/1503 Something that should be considered moving forward is how (or whether) server session sharing needs to be augmented to take the client cert requirements into account. --- If your project

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-03-01 Thread persiaAziz
Github user persiaAziz commented on the issue: https://github.com/apache/trafficserver/pull/1503 It should be merged with master, as well as backported to 7.1.x. Client cert is being checked every time a remap is performed rather it should be checked before making an actual ssl

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-02-28 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1503 Not sure I follow here, does this land on master? Or was the intent here to make a backport PR against 7.1.x branch? --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-02-27 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1503 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/190/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-02-27 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1503 Intel CC build *successful*! See https://ci.trafficserver.apache.org/job/icc-github/58/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-02-27 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1503 Linux build *successful*! See https://ci.trafficserver.apache.org/job/linux-github/1520/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-02-27 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1503 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1625/ for details. --- If your project is set up for it, you can reply to this email and have