prometheus-community / ipmi_exporter

Remote IPMI exporter for Prometheus
MIT License
459 stars 131 forks source link

fix: correct issue after run gosec #170

Closed jplanckeel closed 11 months ago

jplanckeel commented 11 months ago

Context

To improve security, we ran the gosec lint to check for potential security holes in the project.

gosec : https://securego.io/docs/rules/rule-intro

Fix

[x] G104 (CWE-703): Errors unhandled. (Confidence: HIGH, Severity: LOW) [x] G304 (CWE-22): Potential file inclusion via variable (Confidence: HIGH, Severity: MEDIUM) [x] G112 (CWE-400): Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (Confidence: LOW, Severity: MEDIUM)

Change

jplanckeel commented 11 months ago

Hello, why this MR is close ?