rawlinp opened a new pull request #5062: URL: https://github.com/apache/trafficcontrol/pull/5062
## What does this PR (Pull Request) do? - [x] This PR fixes #5019 ## Which Traffic Control components are affected by this PR? - Traffic Ops ## What is the best way to verify this PR? 1. turn on dsRequests in traffic_portal_properties.json and start up TP 2. change an existing ds to utilize a topology and add values for firstHeaderRewrite, innerHeaderRewrite, and lastHeaderRewrite. hit update. 3. a prompt will ask you what status to create the DSR with. choose draft and save and inspect the request sent to POST /deliveryservice_request and ensure that the body of the request contains ds.topology, ds.firstHeaderRewrite, ds.innerHeaderRewrite and ds.lastHeaderRewrite with your values. 4. submit and fulfill the DSR, verify that the changes have been made on the DS. ## If this is a bug fix, what versions of Traffic Control are affected? - master ## The following criteria are ALL met by this PR - [x] Did not include TO API test changes due to the fact that this endpoint is currently being rewritten at the same time this PR was opened. In order to reduce merge conflicts, new TO API tests should be included in the rewrite PR. - [x] bug fix, no docs necessary - [x] unreleased bug, no changelog necessary - [x] This PR includes any and all required license headers - [x] This PR does not include a database migration - [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) ---------------------------------------------------------------- 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]
