trafficcontrol-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [trafficcontrol] 02/02: Changed to include everything up to query params. It was also noticed that if cachekey is not the first plugin in a remap rule then the URI CAN be different. In the case of https the https is actually dropped from the uri and :443 is added, completely changing the key
Date Thu, 06 Sep 2018 15:34:04 GMT
This is an automated email from the ASF dual-hosted git repository.

elsloo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git

commit 4c4c451b852c64c67b553560888a1b5bc476a1cc
Author: Evan Zelkowitz <19699200+ezelkow1@users.noreply.github.com>
AuthorDate: Thu Jul 26 11:36:31 2018 -0600

    Changed to include everything up to query params.  It was also noticed that if cachekey
is not the first plugin in a remap rule then the URI CAN be different. In the case of https
the https is actually dropped from the uri and :443 is added, completely changing the key
---
 traffic_ops/app/lib/UI/DeliveryService.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/traffic_ops/app/lib/UI/DeliveryService.pm b/traffic_ops/app/lib/UI/DeliveryService.pm
index ec250fe..2c34872 100644
--- a/traffic_ops/app/lib/UI/DeliveryService.pm
+++ b/traffic_ops/app/lib/UI/DeliveryService.pm
@@ -1432,7 +1432,7 @@ sub get_qstring_ignore_remap {
 	my $ats_major_version = shift;
 
 	if ($ats_major_version >= 6) {
-		return " \@plugin=cachekey.so \@pparam=--separator= \@pparam=--remove-all-params=true \@pparam=--remove-path=true
\@pparam=--capture-prefix-uri=/https?:\\/\\/([^?]*)/http:\\/\\/\$1/";
+		return " \@plugin=cachekey.so \@pparam=--separator= \@pparam=--remove-all-params=true \@pparam=--remove-path=true
\@pparam=--capture-prefix-uri=/^([^?]*)/$1/";
 	}
 	else {
 		return " \@plugin=cacheurl.so \@pparam=cacheurl_qstring.config";


Mime
View raw message