README.md

@@ -116,11 +116,3 @@ but it is also output to files. The log files can then be fetched to
 #### Version
 
 Add versioning into the app.
-
-### 2024/10/03
-
-Set up the web server with some necessary/nice to have middlewares.
-
-- Recovery, Logger (already included in Default mode)
-- CORS
-- RequestId

configs/howmuch.yml

@@ -20,10 +20,6 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
-dev-mode: true
-
-addr: :8080
-
 db:
   # DB host
   host: 127.0.0.1
@@ -42,6 +38,7 @@ db:
 
 log:
   level: debug
+  development: true
   disalbe-caller: false
   disable-stacktrace: false
   # console or json

go.mod

@@ -5,12 +5,10 @@ go 1.23.1
 require (
 	github.com/fsnotify/fsnotify v1.7.0
 	github.com/gin-gonic/gin v1.10.0
-	github.com/google/uuid v1.6.0
 	github.com/gosuri/uitable v0.0.4
 	github.com/spf13/cobra v1.8.1
 	github.com/spf13/pflag v1.0.5
 	github.com/spf13/viper v1.19.0
-	github.com/stretchr/testify v1.9.0
 	go.uber.org/zap v1.27.0
 )
 
@@ -19,7 +17,6 @@ require (
 	github.com/bytedance/sonic/loader v0.1.1 // indirect
 	github.com/cloudwego/base64x v0.1.4 // indirect
 	github.com/cloudwego/iasm v0.2.0 // indirect
-	github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc // indirect
 	github.com/fatih/color v1.14.1 // indirect
 	github.com/gabriel-vasile/mimetype v1.4.3 // indirect
 	github.com/gin-contrib/sse v0.1.0 // indirect
@@ -40,7 +37,6 @@ require (
 	github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect
 	github.com/modern-go/reflect2 v1.0.2 // indirect
 	github.com/pelletier/go-toml/v2 v2.2.2 // indirect
-	github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
 	github.com/rivo/uniseg v0.2.0 // indirect
 	github.com/sagikazarmark/locafero v0.4.0 // indirect
 	github.com/sagikazarmark/slog-shim v0.1.0 // indirect

go.sum

@@ -36,8 +36,6 @@ github.com/goccy/go-json v0.10.2/go.mod h1:6MelG93GURQebXPDq3khkgXZkazVtN9CRI+MG
 github.com/google/go-cmp v0.5.9 h1:O2Tfq5qg4qc4AmwVlvv0oLiVAGB7enBSJ2x2DqQFi38=
 github.com/google/go-cmp v0.5.9/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
 github.com/google/gofuzz v1.0.0/go.mod h1:dBl0BpW6vV/+mYPU4Po3pmUjxk6FQPldtuIdl/M65Eg=
-github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
-github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
 github.com/gosuri/uitable v0.0.4 h1:IG2xLKRvErL3uhY6e1BylFzG+aJiwQviDDTfOKeKTpY=
 github.com/gosuri/uitable v0.0.4/go.mod h1:tKR86bXuXPZazfOTG1FIzvjIdXzd0mo4Vtn16vt0PJo=
 github.com/hashicorp/hcl v1.0.0 h1:0Anlzjpi4vEasTeNFn2mLJgTSwt0+6sfsiTG8qcWGx4=

internal/howmuch/helpers.go

@@ -28,7 +28,6 @@ import (
 	"strings"
 
 	"git.vinchent.xyz/vinchent/howmuch/internal/pkg/log"
-	"github.com/fsnotify/fsnotify"
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 )
@@ -69,12 +68,6 @@ func initConfig() {
 		return
 	}
 
-	// watching reloading conf
-	viper.OnConfigChange(func(e fsnotify.Event) {
-		log.InfoLog("Config file changed:", e.Name)
-	})
-	viper.WatchConfig()
-
 	log.DebugLog("Using config file", "file", viper.ConfigFileUsed())
 }
 
@@ -82,7 +75,7 @@ func initConfig() {
 func logOptions() *log.Options {
 	return &log.Options{
 		Level:             viper.GetString("log.level"),
-		Development:       viper.GetBool("dev-mode"),
+		Development:       viper.GetBool("log.development"),
 		DisableCaller:     viper.GetBool("log.disable-caller"),
 		DisableStacktrace: viper.GetBool("log.disable-stacktrace"),
 		Format:            viper.GetString("log.format"),

internal/howmuch/howmuch.go

@@ -23,11 +23,13 @@
 package howmuch
 
 import (
+	"encoding/json"
 	"fmt"
 	"net/http"
 
 	"git.vinchent.xyz/vinchent/howmuch/internal/pkg/log"
 	"git.vinchent.xyz/vinchent/howmuch/pkg/version/verflag"
+	"github.com/fsnotify/fsnotify"
 	"github.com/gin-gonic/gin"
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
@@ -87,12 +89,18 @@ to share their expense of an event or a trip`,
 }
 
 func run() error {
-	isDev := viper.GetBool("dev-mode")
+	log.DebugLog("How much do I owe you?")
-	if isDev {
+
-		gin.SetMode(gin.DebugMode)
+	settings, _ := json.MarshalIndent(viper.AllSettings(), "", "  ")
-	} else {
+
-		gin.SetMode(gin.ReleaseMode)
+	// watching reloading conf
-	}
+	viper.OnConfigChange(func(e fsnotify.Event) {
+		log.InfoLog("Config file changed:", e.Name)
+	})
+	viper.WatchConfig()
+
+	log.InfoLog(string(settings))
+	log.InfoLog(viper.GetString("db.username"))
 
 	r := gin.Default()
 
@@ -102,7 +110,7 @@ func run() error {
 		})
 	})
 
-	r.Run(viper.GetString("addr"))
+	r.Run(":8080")
 
 	return nil
 }

internal/pkg/middleware/requestid.go

@@ -1,29 +0,0 @@
-package middleware
-
-import (
-	"github.com/gin-gonic/gin"
-	"github.com/google/uuid"
-)
-
-const requestID = "X-Request-Id"
-
-func RequestID() gin.HandlerFunc {
-	return func(ctx *gin.Context) {
-		var rid string
-
-		if rid = ctx.GetString(requestID); rid != "" {
-			// request id exists already
-			ctx.Next()
-		}
-
-		if rid = ctx.GetHeader(requestID); rid != "" {
-			ctx.Set(requestID, rid)
-			ctx.Next()
-		}
-
-		rid = uuid.NewString()
-		ctx.Set(requestID, rid)
-		ctx.Writer.Header().Add(requestID, rid)
-		ctx.Next()
-	}
-}

internal/pkg/middleware/requestid_test.go

@@ -1,56 +0,0 @@
-package middleware
-
-import (
-	"net/http"
-	"net/http/httptest"
-	"testing"
-
-	"github.com/gin-gonic/gin"
-	"github.com/google/uuid"
-	"github.com/stretchr/testify/assert"
-)
-
-type header struct {
-	Key   string
-	Value string
-}
-
-func performRequest(
-	r http.Handler,
-	method, path string,
-	headers ...header,
-) *httptest.ResponseRecorder {
-	req := httptest.NewRequest(method, path, nil)
-	for _, h := range headers {
-		req.Header.Add(h.Key, h.Value)
-	}
-	w := httptest.NewRecorder()
-	r.ServeHTTP(w, req)
-	return w
-}
-
-func TestRequestID(t *testing.T) {
-	r := gin.New()
-	r.Use(RequestID())
-	var got string
-	wanted := "123"
-
-	r.GET("/example", func(c *gin.Context) {
-		got = c.GetString(requestID)
-		c.Status(http.StatusOK)
-	})
-
-	r.POST("/example", func(c *gin.Context) {
-		got = c.GetString(requestID)
-		c.String(http.StatusAccepted, "ok")
-	})
-
-	// Test with Request ID
-	_ = performRequest(r, "GET", "/example?a=100", header{requestID, wanted})
-	assert.Equal(t, "123", got)
-
-	res := performRequest(r, "GET", "/example?a=100")
-	assert.NotEqual(t, "", got)
-	assert.NoError(t, uuid.Validate(got))
-	assert.Equal(t, res.Header()[requestID][0], got)
-}

pkg/version/verflag/verflag.go

@@ -41,8 +41,8 @@ func AddFlags(fs *pflag.FlagSet) {
 }
 
 func PrintVersion() {
+	fmt.Printf("%s\n", version.Get().String())
 	if *doPrint {
-		fmt.Printf("%s\n", version.Get().String())
 		os.Exit(0)
 	}
 }