You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2020/11/06 13:57:23 UTC
[GitHub] [trafficcontrol] traeak opened a new issue #5254: Add a Mid raw remap text field for delivery services.
traeak opened a new issue #5254:
URL: https://github.com/apache/trafficcontrol/issues/5254
<!--
************ STOP!! ************
If this issue identifies a security vulnerability, DO NOT submit it! Instead, contact
the Apache Traffic Control Security Team at security@trafficcontrol.apache.org and follow the
guidelines at https://www.apache.org/security/ regarding vulnerability disclosure.
- For *SUPPORT QUESTIONS*, use the Traffic Control slack (https://s.apache.org/atc-slack)
or Traffic Control mailing lists (https://trafficcontrol.apache.org/mailing_lists).
- Before submitting, please **SEARCH GITHUB** for a similar issue or PR.
-->
## I'm submitting a ...
<!-- delete all those that don't apply -->
<!--- security vulnerability (STOP!! - see above)-->
- improvement request (usability, performance, tech debt, etc.)
## Traffic Control components affected ...
<!-- delete all those that don't apply -->
- Documentation
- Traffic Ops
- Traffic Ops ORT
- Traffic Portal
- CI tests
## Current behavior:
<!-- Describe how the current features are insufficient. -->
Currently the Delivery Service page in TP has a "Raw Remap Text" field which just applies to the Edge remap rules.
## New behavior:
<!-- Describe how the feature would improve Traffic Control -->
I'd like this to be split into:
Mid Raw Remap Text
and
Edge Raw Remap Text
This should allow experimental plugins and/or plugin options to be used without requiring a new release of TC.
## Minimal reproduction of the problem with instructions:
<!--
If you can illustrate your feature request better with an example, please
provide the *STEPS TO REPRODUCE* and include the applicable TC version.
If not, feel free to delete this section.
-->
## Anything else:
<!--
e.g. additional context, related issues, suggestions about how to approach (feel
free to delete this section)
-->
<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at
https://apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [trafficcontrol] traeak commented on issue #5254: Add a Mid raw remap text field for delivery services.
Posted by GitBox <gi...@apache.org>.
traeak commented on issue #5254:
URL: https://github.com/apache/trafficcontrol/issues/5254#issuecomment-723223588
There needs to be a way to configure ATS without requiring a new version of TC to do it. This particular instance should allow new plugins and plugin options to be exercised.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [trafficcontrol] rawlinp commented on issue #5254: Add a Mid raw remap text field for delivery services.
Posted by GitBox <gi...@apache.org>.
rawlinp commented on issue #5254:
URL: https://github.com/apache/trafficcontrol/issues/5254#issuecomment-723203088
With topologies, the traditional idea of a "mid" is changed. Would we want this raw remap to apply to basically all forward proxy caches? Do we need to differentiate between forward proxies that are _last_ in the hierarchy (just before the origin) and forward proxies that aren't (aka "inner")?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [trafficcontrol] rob05c commented on issue #5254: Add a Mid raw remap text field for delivery services.
Posted by GitBox <gi...@apache.org>.
rob05c commented on issue #5254:
URL: https://github.com/apache/trafficcontrol/issues/5254#issuecomment-723207742
Yeah, going forward this would have to be "inner" or "last" just like Header Rewrites.
But, I think we should think very hard about this and similar things, and see if there's a more structured way to do it.
Injecting raw text like this prone to typos, bugs, ops misunderstandings or not noticing it.
We should really see if there's a safer way to accomplish whatever the specific need here is.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [trafficcontrol] rob05c edited a comment on issue #5254: Add a Mid raw remap text field for delivery services.
Posted by GitBox <gi...@apache.org>.
rob05c edited a comment on issue #5254:
URL: https://github.com/apache/trafficcontrol/issues/5254#issuecomment-723207742
Yeah, going forward this would have to be "inner" or "last" just like Header Rewrites.
But, I think we should think very hard about this and similar things, and see if there's a more structured way to do it.
Injecting raw text like this is prone to typos, bugs, ops misunderstandings or not noticing it.
We should really see if there's a safer way to accomplish whatever the specific need here is.
----------------------------------------------------------------
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:
users@infra.apache.org