fix: ensure proper application reload on configuration change and reg…

…ister Prometheus metrics once

api/v2/echo.go

@@ -29,6 +29,8 @@ type customValidator struct {
 	validator *validator.Validate
 }
 
+var registerMetricsMiddleware sync.Once
+
 func (cv *customValidator) Validate(i interface{}) error {
 	if err := cv.validator.Struct(i); err != nil {
 		return echo.NewHTTPError(http.StatusBadRequest, err.Error())
@@ -64,9 +66,10 @@ func V2() (*echo.Echo, error) {
 
 	e.Validator = &customValidator{validator: validator.New()}
 
-	// Prometheus middleware
-	e.Use(echoprometheus.NewMiddleware("myapp"))
-	e.GET("/metrics", echoprometheus.NewHandler())
+	registerMetricsMiddleware.Do(func() {
+		e.Use(echoprometheus.NewMiddleware("myapp"))
+		e.GET("/metrics", echoprometheus.NewHandler())
+	})
 
 	s := &server{}
 	l := linkedlist.NewLinkedList()

config/configs.go

@@ -14,9 +14,9 @@ var MapLevel = map[string]slog.Level{
 	"ERROR": slog.LevelError,
 }
 
-var Confs config
+var Confs Config
 
-type config struct {
+type Config struct {
 	Server server `yaml:"server"`
 	Logger logger `yaml:"logger"`
 }

main.go

@@ -34,20 +34,24 @@ func run(ctx context.Context) (*api.Api, error) {
 
 	server, err := api.New()
 	if err != nil {
-		slog.Error("Bootin api", "error", err)
+		slog.Error("Booting api", "error", err)
 		return nil, err
 	}
 
 	go func() {
 		if err := server.Start(ctx); err != nil && !errors.Is(err, http.ErrServerClosed) {
-			slog.Error("Starting server", "errror", err)
+			slog.Error("Starting server", "error", err)
 			os.Exit(1)
 		}
 	}()
 
 	return server, nil
 }
 
+func configChanged(oldConfig *config.Config) bool {
+	return oldConfig.Server.Port != config.Confs.Server.Port || oldConfig.Logger.Level != config.Confs.Logger.Level
+}
+
 func main() {
 	flag.Parse()
 
@@ -65,10 +69,23 @@ func main() {
 		sig := <-sigs
 		switch sig {
 		case syscall.SIGHUP:
-			slog.Info("Received SIGHUP, reloading configuration...")
+			slog.Info("Received SIGHUP, checking configuration...")
 
-			err = server.Shutdown(ctx)
-			if err != nil {
+			oldConfig := config.Confs
+
+			if err := config.Load(*cfg); err != nil {
+				slog.Error("Reading new configuration", "error", err)
+				continue
+			}
+
+			if !configChanged(&oldConfig) {
+				slog.Info("Configuration unchanged, no reload needed.")
+				continue
+			}
+
+			slog.Info("Configuration has changed, reloading server...")
+
+			if err = server.Shutdown(ctx); err != nil {
 				slog.Error("could not stop server", "error", err)
 				continue
 			}
@@ -87,5 +104,4 @@ func main() {
 			os.Exit(0)
 		}
 	}
-
 }