Merge pull request #298 from authorizerdev/fix/url-parsing

lakhansamani · web-flow · commit c948c98e9428 · 2022-11-24T19:13:50.000+05:30
fix: remove extra slash from host
diff --git a/server/parsers/url.go b/server/parsers/url.go
@@ -20,20 +20,20 @@ func GetHost(c *gin.Context) string {
 		authorizerURL = ""
 	}
 	if authorizerURL != "" {
-		return authorizerURL
+		return strings.TrimSuffix(authorizerURL, "/")
 	}
 
 	authorizerURL = c.Request.Header.Get("X-Authorizer-URL")
 	if authorizerURL != "" {
-		return authorizerURL
+		return strings.TrimSuffix(authorizerURL, "/")
 	}
 
 	scheme := c.Request.Header.Get("X-Forwarded-Proto")
 	if scheme != "https" {
 		scheme = "http"
 	}
 	host := c.Request.Host
-	return scheme + "://" + host
+	return strings.TrimSuffix(scheme+"://"+host, "/")
 }
 
 // GetHostName function returns hostname and port

Original file line number	Diff line number	Diff line change
`@@ -20,20 +20,20 @@ func GetHost(c *gin.Context) string {`
`20`	`20`	`authorizerURL = ""`
`21`	`21`	`}`
`22`	`22`	`if authorizerURL != "" {`
`23`		`- return authorizerURL`
	`23`	`+ return strings.TrimSuffix(authorizerURL, "/")`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`authorizerURL = c.Request.Header.Get("X-Authorizer-URL")`
`27`	`27`	`if authorizerURL != "" {`
`28`		`- return authorizerURL`
	`28`	`+ return strings.TrimSuffix(authorizerURL, "/")`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`scheme := c.Request.Header.Get("X-Forwarded-Proto")`
`32`	`32`	`if scheme != "https" {`
`33`	`33`	`scheme = "http"`
`34`	`34`	`}`
`35`	`35`	`host := c.Request.Host`
`36`		`- return scheme + "://" + host`
	`36`	`+ return strings.TrimSuffix(scheme+"://"+host, "/")`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`// GetHostName function returns hostname and port`