Github user PSUdaemon commented on the pull request:
https://github.com/apache/trafficserver/pull/330#issuecomment-208439499
Closing this per discussion above.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your proje
Github user PSUdaemon closed the pull request at:
https://github.com/apache/trafficserver/pull/330
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the featu
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/330#issuecomment-202092069
I think for now at least, we should close this, the potential negative side
effects severely outweighs the positive (and hopefully the header_rewrite
workaround wor
Github user bryancall commented on the pull request:
https://github.com/apache/trafficserver/pull/330#issuecomment-187811485
We talked about this at the github meeting. As a workaround you can adjust
the Last-Modified with header rewrite. We feel this is the best solution to
only cha
Github user ngorchilov commented on the pull request:
https://github.com/apache/trafficserver/pull/330#issuecomment-158017177
@zwoop: here you are:
http://sshds5-lh.akamaihd.net/z/delayed/indvsa2015_INDVSSATEST1DAY1_1@336263/464_209823ecd2922291-p.bootstrap?g=YCEAMIWDQZQT&hdntl=exp
Github user PSUdaemon commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r45298596
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/330#issuecomment-157917453
Can you provide some example URLs where this fails proxying through ATS?
I'm trying t both reproduce and engage some friends over at Akamai, to fully
understand thi
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r44595605
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r44594400
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Hi,
I've reported the same issue some time ago, in the jira TS-3212 you can find my
workaround.
Github user ngorchilov commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r44505183
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r44432922
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Github user ngorchilov commented on the pull request:
https://github.com/apache/trafficserver/pull/330#issuecomment-155321872
@jpeach not exactly sure how to implement that from scratch. is there any
existing test that can be patched to do cover this cast too?
---
If your project is
Github user ngorchilov commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r44374354
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Github user ngorchilov commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r44374071
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/330#discussion_r44372265
--- Diff: proxy/http/HttpTransactCache.cc ---
@@ -1299,7 +1299,7 @@
HttpTransactCache::match_response_to_request_conditionals(HTTPHdr *request, HTTP
Github user jpeach commented on the pull request:
https://github.com/apache/trafficserver/pull/330#issuecomment-155227320
Could you also add a test for this? Otherwise it is very likely to regress
in the future.
---
If your project is set up for it, you can reply to this email and ha
17 matches
Mail list logo
