fix auth issue with vLLM

bricks-cloud · Apr 9, 2024 · 65b1653 · 65b1653
1 parent 524a2e8
commit 65b1653
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,8 @@
+
+## 1.15.2 - 2024-04-09
+### Fixed
+- Fixed an issue with `apikey` being required for making vLLM requests
+
 ## 1.15.1 - 2024-04-07
 ### Added
 - Added support for `apikey` in vLLM integration

diff --git a/internal/authenticator/authenticator.go b/internal/authenticator/authenticator.go
@@ -78,8 +78,10 @@ func rewriteHttpAuthHeader(req *http.Request, setting *provider.Setting) error {
 
 	apiKey := setting.GetParam("apikey")
 
-	if strings.HasPrefix(uri, "/api/providers/vllm") && len(apiKey) != 0 {
-		req.Header.Set("Authorization", fmt.Sprintf("Bearer %s", apiKey))
+	if strings.HasPrefix(uri, "/api/providers/vllm") {
+		if len(apiKey) != 0 {
+			req.Header.Set("Authorization", fmt.Sprintf("Bearer %s", apiKey))
+		}
 		return nil
 	}