Skip to content

Commit aae072b

Browse files
leodidoona-agent
andcommitted
fix(lint): remove unused function and fix unchecked error returns
- Remove unused getEnvOrDefault function - Add blank identifier assignments for unchecked error returns in test file Co-authored-by: Ona <[email protected]>
1 parent d7ccd50 commit aae072b

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

pkg/leeway/signing/attestation.go

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -426,11 +426,3 @@ func fetchGitHubOIDCToken(ctx context.Context, audience string) (string, error)
426426

427427
return payload.Value, nil
428428
}
429-
430-
// getEnvOrDefault returns environment variable value or default
431-
func getEnvOrDefault(key, defaultValue string) string {
432-
if value := os.Getenv(key); value != "" {
433-
return value
434-
}
435-
return defaultValue
436-
}

pkg/leeway/signing/attestation_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1117,7 +1117,7 @@ func TestFetchGitHubOIDCToken(t *testing.T) {
11171117
t.Error("Missing or invalid Authorization header")
11181118
}
11191119
w.WriteHeader(http.StatusOK)
1120-
json.NewEncoder(w).Encode(map[string]string{"value": "test-token-12345"})
1120+
_ = json.NewEncoder(w).Encode(map[string]string{"value": "test-token-12345"})
11211121
}))
11221122
},
11231123
audience: "sigstore",
@@ -1138,7 +1138,7 @@ func TestFetchGitHubOIDCToken(t *testing.T) {
11381138
mockServer: func(t *testing.T) *httptest.Server {
11391139
return httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
11401140
w.WriteHeader(http.StatusInternalServerError)
1141-
w.Write([]byte(`{"error": "internal error"}`))
1141+
_, _ = w.Write([]byte(`{"error": "internal error"}`))
11421142
}))
11431143
},
11441144
audience: "sigstore",
@@ -1150,7 +1150,7 @@ func TestFetchGitHubOIDCToken(t *testing.T) {
11501150
mockServer: func(t *testing.T) *httptest.Server {
11511151
return httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
11521152
w.WriteHeader(http.StatusOK)
1153-
json.NewEncoder(w).Encode(map[string]string{"value": ""})
1153+
_ = json.NewEncoder(w).Encode(map[string]string{"value": ""})
11541154
}))
11551155
},
11561156
audience: "sigstore",
@@ -1162,7 +1162,7 @@ func TestFetchGitHubOIDCToken(t *testing.T) {
11621162
mockServer: func(t *testing.T) *httptest.Server {
11631163
return httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
11641164
w.WriteHeader(http.StatusOK)
1165-
w.Write([]byte(`invalid json`))
1165+
_, _ = w.Write([]byte(`invalid json`))
11661166
}))
11671167
},
11681168
audience: "sigstore",

0 commit comments

Comments
 (0)