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 2021/04/01 15:52:26 UTC
[GitHub] [trafficcontrol] traeak opened a new issue #5698: Traffic Portal Invalidate Content page: warning about origin needing to support 304s
traeak opened a new issue #5698:
URL: https://github.com/apache/trafficcontrol/issues/5698
<!--
************ 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 -->
- Traffic Portal
## Current behavior:
<!-- Describe how the current features are insufficient. -->
Recently we issued a large scale invalidation request against one of our delivery services. Unfortunately we didn't realize that the origin was not configured to handle If-Modified-Since or If-None-Match headers and always sent back 200s instead of 304s. This resulted in a storm of requests going to the origin, and a new mini storm starting every time an edge cache ATS was restarted.
Perhaps some message or some documentation should be added to ensure that whoever uses Invalidations realizes that they are designed to be used with origins that properly handle IMS requests.
## New behavior:
<!-- Describe how the feature would improve Traffic Control -->
## 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] TaylorCFrey commented on issue #5698: Traffic Portal Invalidate Content page: warning about origin needing to support 304s
Posted by GitBox <gi...@apache.org>.
TaylorCFrey commented on issue #5698:
URL: https://github.com/apache/trafficcontrol/issues/5698#issuecomment-847906187
Throw this one on my docket!
--
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] mitchell852 edited a comment on issue #5698: Traffic Portal Invalidate Content page: warning about origin needing to support 304s
Posted by GitBox <gi...@apache.org>.
mitchell852 edited a comment on issue #5698:
URL: https://github.com/apache/trafficcontrol/issues/5698#issuecomment-812009311
Seems easy enough to create a warning in both of these places in TP:
https://tp.domain.tld/#!/jobs/new
https://tp.domain.tld/#!/delivery-services/{{id}}/jobs?type={{ds_type}}
something along the lines of "Delivery service origin(s) must support IMS in order to safely invalidate content"
Also, that should be added to the documentation:
https://traffic-control-cdn.readthedocs.io/en/latest/admin/quick_howto/content_invalidation.html
https://traffic-control-cdn.readthedocs.io/en/latest/admin/traffic_portal/usingtrafficportal.html
and anywhere else in the documentation that discusses content invalidation.
--
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] mitchell852 commented on issue #5698: Traffic Portal Invalidate Content page: warning about origin needing to support 304s
Posted by GitBox <gi...@apache.org>.
mitchell852 commented on issue #5698:
URL: https://github.com/apache/trafficcontrol/issues/5698#issuecomment-812009311
Seems easy enough to create a warning in both of these places in TP:
https://tp.domain.tld/#!/jobs/new
https://tp.domain.tld/#!/delivery-services/{{id}}/jobs?type={{ds_type}}
something along the lines of "Delivery service origin must support IMS in order to safely invalidate content"
Also, that should be added to the documentation:
https://traffic-control-cdn.readthedocs.io/en/latest/admin/quick_howto/content_invalidation.html
https://traffic-control-cdn.readthedocs.io/en/latest/admin/traffic_portal/usingtrafficportal.html
and anywhere else in the documentation that discusses content invalidation.
--
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] mattjackson220 closed issue #5698: Traffic Portal Invalidate Content page: warning about origin needing to support 304s
Posted by GitBox <gi...@apache.org>.
mattjackson220 closed issue #5698:
URL: https://github.com/apache/trafficcontrol/issues/5698
--
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.
To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org