rob05c opened a new pull request #5657: URL: https://github.com/apache/trafficcontrol/pull/5657
Fixes the TC Service Category header to be internal to ATS, and not sent by ATS in HTTP requests or responses. Docs already exist for Service Category. No changelog, no interface change. Tested by existing Service Category Header Rewrite tests. - [x] This PR fixes #REPLACE_ME OR is not related to any Issue <!-- You can check for an issue here: https://github.com/apache/trafficcontrol/issues --> ## Which Traffic Control components are affected by this PR? - Traffic Ops ORT ## What is the best way to verify this PR? Run unit tests. Generate config, verify header rewrites are prefixed with `@` to make them internal to ATS. Run ATS, verify header writes are not passed upstream or downstream and are logged if an appropriate `svc=%<{@CDN_SVC}cqh>` log field exists. ## If this is a bug fix, what versions of Traffic Control are affected? `master` ## The following criteria are ALL met by this PR - [x] This PR includes tests OR I have explained why tests are unnecessary - [x] This PR includes documentation OR I have explained why documentation is unnecessary - [x] This PR includes an update to CHANGELOG.md OR such an update is not necessary - [x] This PR includes any and all required license headers - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY** (see [the Apache Software Foundation's security guidelines](https://www.apache.org/security/) for details) ## Additional Information -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
