Fix TO to remove proxy route for Go route#4210
Conversation
|
Refer to this link for build results (access rights to CI server needed): |
There was a problem hiding this comment.
Oy, the route ordering strikes again. I think this route was here because it otherwise gets eaten by the less specific GET /cdns/{id} route. So the new GET /cdns/health routes need to be moved up here as well.
There was a problem hiding this comment.
Ah, I'll fix. Give me a bit, currently using my machine to diff configs, planning to test this when that's done.
rob05c
force-pushed
the
remove-to-proxy-route
branch
from
caf4287 to
4500b32
Compare
Removes a proxyHandler route, for an endpoint which was rewritten in Go and has a Go route.
rob05c
force-pushed
the
remove-to-proxy-route
branch
from
4500b32 to
f0fd1b3
Compare
|
Refer to this link for build results (access rights to CI server needed): |
|
Refer to this link for build results (access rights to CI server needed): |
3 participants
Removes a proxyHandler route, for an endpoint which was rewritten in
Go and has a Go route.
See https://github.com/apache/trafficcontrol/pull/2305/files
No tests, just removing a proxy, existing route has been manually tested.
No docs, no interface change.
No changelog, no interface change.
Which Traffic Control components are affected by this PR?
What is the best way to verify this PR?
Request cdns/health route, verify matches Perl, and has
x-server-nameGolang header.If this is a bug fix, what versions of Traffic Control are affected?
Not a bug fix.
The following criteria are ALL met by this PR
Additional Information