You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@knox.apache.org by km...@apache.org on 2014/10/10 22:51:22 UTC

git commit: KNOX-439: URL pattern matching fails for default ports HTTP 80 and HTTPS 443

Repository: knox
Updated Branches:
  refs/heads/master 974312ead -> b40aa7881


KNOX-439: URL pattern matching fails for default ports HTTP 80 and HTTPS 443


Project: http://git-wip-us.apache.org/repos/asf/knox/repo
Commit: http://git-wip-us.apache.org/repos/asf/knox/commit/b40aa788
Tree: http://git-wip-us.apache.org/repos/asf/knox/tree/b40aa788
Diff: http://git-wip-us.apache.org/repos/asf/knox/diff/b40aa788

Branch: refs/heads/master
Commit: b40aa7881697645ce9dca5f167ce12bfa91c4d59
Parents: 974312e
Author: Kevin Minder <ke...@hortonworks.com>
Authored: Fri Oct 10 16:51:17 2014 -0400
Committer: Kevin Minder <ke...@hortonworks.com>
Committed: Fri Oct 10 16:51:17 2014 -0400

----------------------------------------------------------------------
 .../gateway/filter/rewrite/impl/UrlRewriteRequest.java | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/knox/blob/b40aa788/gateway-provider-rewrite/src/main/java/org/apache/hadoop/gateway/filter/rewrite/impl/UrlRewriteRequest.java
----------------------------------------------------------------------
diff --git a/gateway-provider-rewrite/src/main/java/org/apache/hadoop/gateway/filter/rewrite/impl/UrlRewriteRequest.java b/gateway-provider-rewrite/src/main/java/org/apache/hadoop/gateway/filter/rewrite/impl/UrlRewriteRequest.java
index cbe6806..e97d8e1 100644
--- a/gateway-provider-rewrite/src/main/java/org/apache/hadoop/gateway/filter/rewrite/impl/UrlRewriteRequest.java
+++ b/gateway-provider-rewrite/src/main/java/org/apache/hadoop/gateway/filter/rewrite/impl/UrlRewriteRequest.java
@@ -76,8 +76,17 @@ public class UrlRewriteRequest extends GatewayRequestWrapper implements Resolver
   }
 
   private Template getSourceUrl() {
-    Template urlTemplate = null;
-    StringBuffer urlString = super.getRequestURL();
+    Template urlTemplate;
+    //KNOX-439[
+    //StringBuffer urlString = super.getRequestURL();
+    StringBuffer urlString = new StringBuffer( 128 );
+    urlString.append( getScheme() );
+    urlString.append( "://" );
+    urlString.append( getServerName() );
+    urlString.append( ":" );
+    urlString.append( getServerPort() );
+    urlString.append( super.getRequestURI() );
+    //]
     String queryString = super.getQueryString();
     if( queryString != null ) {
       urlString.append( '?' );