Skip to content

Commit 70eb80d

Browse files
weltekialexellis
authored andcommitted
Fix url for remote builder
Only keep scheme and host when initialzing the builder client. Signed-off-by: Han Verstraete (OpenFaaS Ltd) <[email protected]>
1 parent 3da8558 commit 70eb80d

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

builder/build.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,11 @@ func BuildImage(image string, handler string, functionName string, language stri
112112
payloadSecret = bytes.TrimSpace(payloadSecret)
113113

114114
// Initialize a new builder client.
115-
builderURL, _ := url.Parse(remoteBuilder)
115+
u, _ := url.Parse(remoteBuilder)
116+
builderURL := &url.URL{
117+
Scheme: u.Scheme,
118+
Host: u.Host,
119+
}
116120
b := builder.NewFunctionBuilder(builderURL, http.DefaultClient, builder.WithHmacAuth(string(payloadSecret)))
117121

118122
stream, err := b.BuildWithStream(tarPath)

builder/publish.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,11 @@ func PublishImage(image string, handler string, functionName string, language st
104104
payloadSecret = bytes.TrimSpace(payloadSecret)
105105

106106
// Initialize a new builder client.
107-
builderURL, _ := url.Parse(remoteBuilder)
107+
u, _ := url.Parse(remoteBuilder)
108+
builderURL := &url.URL{
109+
Scheme: u.Scheme,
110+
Host: u.Host,
111+
}
108112
b := builder.NewFunctionBuilder(builderURL, http.DefaultClient, builder.WithHmacAuth(string(payloadSecret)))
109113

110114
stream, err := b.BuildWithStream(tarPath)

0 commit comments

Comments
 (0)