Temporarily disable location header normalization

pulumi · Aug 29, 2024 · ff8d746 · ff8d746
1 parent 21058dc
commit ff8d746
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 11 deletions.
diff --git a/provider/pkg/azure/client_azcore.go b/provider/pkg/azure/client_azcore.go
@@ -214,17 +214,17 @@ func normalizeLocationHeader(host, apiVersion string, headers http.Header) {
 		return
 	}
 
-	if !locUrl.IsAbs() {
-		locUrl.Host = host
-		absUrlStr := runtime.JoinPaths(host, locUrlStr)
-		absUrl, err := url.Parse(absUrlStr)
-		if err != nil {
-			logging.V(3).Infof("Location header '%s' is not an absolute URL, failed to make absolute: %s", locUrlStr, absUrlStr)
-		} else {
-			locUrl = absUrl
-			logging.V(9).Infof("Location header '%s' is not an absolute URL, added host '%s'", locUrlStr, host)
-		}
-	}
+	// if !locUrl.IsAbs() {
+	// 	locUrl.Host = host
+	// 	absUrlStr := runtime.JoinPaths(host, locUrlStr)
+	// 	absUrl, err := url.Parse(absUrlStr)
+	// 	if err != nil {
+	// 		logging.V(3).Infof("Location header '%s' is not an absolute URL, failed to make absolute: %s", locUrlStr, absUrlStr)
+	// 	} else {
+	// 		locUrl = absUrl
+	// 		logging.V(9).Infof("Location header '%s' is not an absolute URL, added host '%s'", locUrlStr, host)
+	// 	}
+	// }
 
 	if locUrl.Query().Get("api-version") == "" {
 		q := locUrl.Query()

diff --git a/provider/pkg/azure/client_azcore_test.go b/provider/pkg/azure/client_azcore_test.go
@@ -16,6 +16,7 @@ import (
 )
 
 func TestNormalizeLocationHeader(t *testing.T) {
+	t.Skip("TODO,tkappler")
 	const host = "https://management.azure.com"
 	const apiVersion = "2022-09-01"