Skip to content

Commit 10f5fc7

Browse files
authored
azure-metrics-exporter: fips-compliant sha256 replacement (#126)
1 parent 7f087ff commit 10f5fc7

File tree

6 files changed

+12
-12
lines changed

6 files changed

+12
-12
lines changed

metrics/servicediscovery.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package metrics
22

33
import (
44
"context"
5-
"crypto/sha1" // #nosec G505
5+
"crypto/sha256"
66
"encoding/json"
77
"fmt"
88
"log/slog"
@@ -41,7 +41,7 @@ func (sd *AzureServiceDiscovery) fetchResourceList(subscriptionId, filter string
4141
// nolint:gosec
4242
cacheKey := fmt.Sprintf(
4343
"%x",
44-
sha1.Sum([]byte(fmt.Sprintf("%v:%v", subscriptionId, filter))),
44+
sha256.Sum256([]byte(fmt.Sprintf("%v:%v", subscriptionId, filter))),
4545
)
4646

4747
// try to fetch info from cache

probe_metrics_list.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package main
22

33
import (
44
"context"
5-
"crypto/sha1" // #nosec G505
5+
"crypto/sha256"
66
"fmt"
77
"log/slog"
88
"net/http"
@@ -54,7 +54,7 @@ func probeMetricsListHandler(w http.ResponseWriter, r *http.Request) {
5454
prober.SetAzureResourceTagManager(AzureResourceTagManager)
5555
prober.SetPrometheusRegistry(registry)
5656
if settings.Cache != nil {
57-
cacheKey := fmt.Sprintf("list:%x", sha1.Sum([]byte(r.URL.String()))) // #nosec G401
57+
cacheKey := fmt.Sprintf("list:%x", sha256.Sum256([]byte(r.URL.String())))
5858
prober.EnableMetricsCache(metricsCache, cacheKey, settings.CacheDuration(startTime))
5959
}
6060

probe_metrics_resource.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package main
22

33
import (
44
"context"
5-
"crypto/sha1" // #nosec G505
5+
"crypto/sha256"
66
"fmt"
77
"log/slog"
88
"net/http"
@@ -54,7 +54,7 @@ func probeMetricsResourceHandler(w http.ResponseWriter, r *http.Request) {
5454
prober.SetAzureResourceTagManager(AzureResourceTagManager)
5555
prober.SetPrometheusRegistry(registry)
5656
if settings.Cache != nil {
57-
cacheKey := fmt.Sprintf("resource:%x", sha1.Sum([]byte(r.URL.String()))) // #nosec G401
57+
cacheKey := fmt.Sprintf("resource:%x", sha256.Sum256([]byte(r.URL.String())))
5858
prober.EnableMetricsCache(metricsCache, cacheKey, settings.CacheDuration(startTime))
5959
}
6060

probe_metrics_resourcegraph.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package main
22

33
import (
44
"context"
5-
"crypto/sha1" // #nosec G505
5+
"crypto/sha256"
66
"fmt"
77
"log/slog"
88
"net/http"
@@ -61,7 +61,7 @@ func probeMetricsResourceGraphHandler(w http.ResponseWriter, r *http.Request) {
6161
prober.SetAzureResourceTagManager(AzureResourceTagManager)
6262
prober.SetPrometheusRegistry(registry)
6363
if settings.Cache != nil {
64-
cacheKey := fmt.Sprintf("scrape:%x", sha1.Sum([]byte(r.URL.String()))) // #nosec G401
64+
cacheKey := fmt.Sprintf("scrape:%x", sha256.Sum256([]byte(r.URL.String())))
6565
prober.EnableMetricsCache(metricsCache, cacheKey, settings.CacheDuration(startTime))
6666
}
6767

probe_metrics_scrape.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package main
22

33
import (
44
"context"
5-
"crypto/sha1" // #nosec G505
5+
"crypto/sha256"
66
"fmt"
77
"log/slog"
88
"net/http"
@@ -66,7 +66,7 @@ func probeMetricsScrapeHandler(w http.ResponseWriter, r *http.Request) {
6666
prober.SetAzureResourceTagManager(AzureResourceTagManager)
6767
prober.SetPrometheusRegistry(registry)
6868
if settings.Cache != nil {
69-
cacheKey := fmt.Sprintf("scrape:%x", sha1.Sum([]byte(r.URL.String()))) // #nosec G401
69+
cacheKey := fmt.Sprintf("scrape:%x", sha256.Sum256([]byte(r.URL.String())))
7070
prober.EnableMetricsCache(metricsCache, cacheKey, settings.CacheDuration(startTime))
7171
}
7272

probe_metrics_subscription.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package main
22

33
import (
44
"context"
5-
"crypto/sha1" // #nosec G505
5+
"crypto/sha256"
66
"fmt"
77
"log/slog"
88
"net/http"
@@ -54,7 +54,7 @@ func probeMetricsSubscriptionHandler(w http.ResponseWriter, r *http.Request) {
5454
prober.SetAzureResourceTagManager(AzureResourceTagManager)
5555
prober.SetPrometheusRegistry(registry)
5656
if settings.Cache != nil {
57-
cacheKey := fmt.Sprintf("list:%x", sha1.Sum([]byte(r.URL.String()))) // #nosec G401
57+
cacheKey := fmt.Sprintf("list:%x", sha256.Sum256([]byte(r.URL.String())))
5858
prober.EnableMetricsCache(metricsCache, cacheKey, settings.CacheDuration(startTime))
5959
}
6060

0 commit comments

Comments
 (0)