Don't use HOME env in the my-cnf config

Use a relative file as the default for the .my.cnf style configuration. Fixes: #633 Signed-off-by: SuperQ <superq@gmail.com>
prometheus · Jun 19, 2023 · 875660e · 875660e
1 parent 2b7699b
commit 875660e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,8 +2,12 @@
 
 BREAKING CHANGES:
 
+Don't prepend the `.my.cnf` file with the `HOME` environment.
+
 Changes:
 
+* [CHANGE] Don't use HOME env in the my-cnf config path. #635
+
 * [CHANGE]
 * [FEATURE]
 * [ENHANCEMENT]

diff --git a/mysqld_exporter.go b/mysqld_exporter.go
@@ -17,7 +17,6 @@ import (
 	"context"
 	"net/http"
 	"os"
-	"path"
 	"strconv"
 	"time"
 
@@ -48,7 +47,7 @@ var (
 	configMycnf = kingpin.Flag(
 		"config.my-cnf",
 		"Path to .my.cnf file to read MySQL credentials from.",
-	).Default(path.Join(os.Getenv("HOME"), ".my.cnf")).String()
+	).Default(".my.cnf").String()
 	mysqldAddress = kingpin.Flag(
 		"mysqld.address",
 		"Address to use for connecting to MySQL",