zrhoffman commented on a change in pull request #4537: Add blueprint for Flexible Topologies URL: https://github.com/apache/trafficcontrol/pull/4537#discussion_r408915906
########## File path: blueprints/flexible-topologies.md ########## @@ -250,11 +250,29 @@ response JSON: ##### `/deliveryservices` endpoints -All relevant Delivery Service APIs will have their JSON request and response objects modified to include a new `topology` field which references the name of the topology it's assigned to: -```JSON +All relevant Delivery Service APIs will have their JSON request and response objects modified to include the following new fields: +- `topology` - the name of the topology it's assigned to +- `firstHeaderRewrite` - the header_rewrite ATS config that will be applied to the *first tier* caches in the Delivery Service's Topology +- `middleHeaderRewrite` - the header_rewrite ATS config that will be applied to all *middle tier* caches in this Delivery Service's Topology +- `lastHeaderRewrite` - the header_rewrite ATS config that will be applied to all *last tier* caches in this Delivery Service's Topology Review comment: IMO `midHeaderRewrite` sounds too close to the existing `midHeaderRewrite` rules that we have today. It would be nice if `firstHeaderRewrite`/`lastHeaderRewrite` were named something that implies their place in the topology, like `leafHeaderRewrite`/`rootNodeHeaderRewrite`. ---------------------------------------------------------------- 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] With regards, Apache Git Services
