{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/4160","labels_url":"https://api.github.com/repos/openshift/origin/issues/4160/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/4160/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/4160/events","html_url":"https://github.com/openshift/origin/pull/4160","id":100905211,"number":4160,"title":"Fix reencrypt routes and add path based support","user":{"login":"ramr","id":928868,"avatar_url":"https://avatars.githubusercontent.com/u/928868?v=3","gravatar_id":"","url":"https://api.github.com/users/ramr","html_url":"https://github.com/ramr","followers_url":"https://api.github.com/users/ramr/followers","following_url":"https://api.github.com/users/ramr/following{/other_user}","gists_url":"https://api.github.com/users/ramr/gists{/gist_id}","starred_url":"https://api.github.com/users/ramr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ramr/subscriptions","organizations_url":"https://api.github.com/users/ramr/orgs","repos_url":"https://api.github.com/users/ramr/repos","events_url":"https://api.github.com/users/ramr/events{/privacy}","received_events_url":"https://api.github.com/users/ramr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-08-14T01:07:49Z","updated_at":"2015-08-19T20:16:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/origin/pulls/4160","html_url":"https://github.com/openshift/origin/pull/4160","diff_url":"https://github.com/openshift/origin/pull/4160.diff","patch_url":"https://github.com/openshift/origin/pull/4160.patch"},"body":"Get reencrypt routes working + add support for path based reencrypt routes - similar to the other http/edge path based checks. This is the same functionality #4097 was going to fix.\r\nThis fixes issue #4011\r\n\r\nWhilst testing this, I did find some issues with ACLs (```map_beg``` doesn't seem to work inside an ACL), so I inlined the ACL code to get it to work and then it further got optimized because the check/condition in  \r\n```use_backend be_secure_%[base,map_beg(/var/lib/haproxy/conf/os_reencrypt.map)] if { base,map_beg(/var/lib/haproxy/conf/os_reencrypt.map) -m found }``` was redundant and so it morphed into its current form. LMK if that's outta whack - it does work fine.\r\n\r\nTest with:  \r\n```\r\nfunction test_uris() {  \r\n   local verbosity=\"\"   #  \"-vvv\"  \r\n   local rip=\"10.245.2.3\"  # \"10.0.2.15\"\r\n   for uri in https://www.example2.com/test/ https://www.example2.com/; do   \r\n     echo \"uri: $uri -> $(curl -s -w \"%{http_code}\" --resolve www.example2.com:443:$rip --cacert _qatests/hello-nginx-docker/certs/mypersonalca/certs/ca.pem $verbosity \"$uri\")\"  \r\n  done  \r\n}   \r\n  \r\noc create -f https://raw.githubusercontent.com/openshift-qe/v3-testfiles/master/routing/nginx-pod.json  \r\noc create -f https://raw.githubusercontent.com/openshift-qe/v3-testfiles/master/routing/reencrypt/service_secure.json  \r\ntest_uris  \r\n  \r\noc create -f https://raw.githubusercontent.com/openshift-qe/v3-testfiles/master/routing/reencrypt/route_reencrypt-path.json  \r\ntest_uris  \r\n  \r\noc create -f https://raw.githubusercontent.com/openshift-qe/v3-testfiles/master/routing/reencrypt/route_reencrypt.json  \r\ntest_uris  \r\n  \r\noc delete -f https://raw.githubusercontent.com/openshift-qe/v3-testfiles/master/routing/reencrypt/route_reencrypt-path.json  \r\ntest_uris  \r\n```\r\n\r\n@pweil- @rajatchopra  PTAL Thx"},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/132767613","html_url":"https://github.com/openshift/origin/pull/4160#issuecomment-132767613","issue_url":"https://api.github.com/repos/openshift/origin/issues/4160","id":132767613,"user":{"login":"openshift-bot","id":1779249,"avatar_url":"https://avatars.githubusercontent.com/u/1779249?v=3","gravatar_id":"","url":"https://api.github.com/users/openshift-bot","html_url":"https://github.com/openshift-bot","followers_url":"https://api.github.com/users/openshift-bot/followers","following_url":"https://api.github.com/users/openshift-bot/following{/other_user}","gists_url":"https://api.github.com/users/openshift-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/openshift-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openshift-bot/subscriptions","organizations_url":"https://api.github.com/users/openshift-bot/orgs","repos_url":"https://api.github.com/users/openshift-bot/repos","events_url":"https://api.github.com/users/openshift-bot/events{/privacy}","received_events_url":"https://api.github.com/users/openshift-bot/received_events","type":"User","site_admin":false},"created_at":"2015-08-19T20:16:49Z","updated_at":"2015-08-19T20:16:49Z","body":"Evaluated for origin merge up to 47216b92afde1c100b7733d4e60e38735faf0415"}}