[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 needed when ATS actually creates an HTTPS connection with the origin --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 connection with the origin server --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---