fix: make create session works
This commit is contained in:
parent
7ff91bab1d
commit
9290bcf88c
@ -7,7 +7,7 @@ tmp_dir = "tmp"
|
|||||||
bin = "./_output/howmuch"
|
bin = "./_output/howmuch"
|
||||||
cmd = "make build"
|
cmd = "make build"
|
||||||
delay = 1000
|
delay = 1000
|
||||||
exclude_dir = ["assets", "tmp", "vendor", "testdata", "_output"]
|
exclude_dir = ["assets", "tmp", "vendor", "testdata", "_output", "internal/howmuch/adapter/repo/sqlc"]
|
||||||
exclude_file = []
|
exclude_file = []
|
||||||
exclude_regex = ["_test.go"]
|
exclude_regex = ["_test.go"]
|
||||||
exclude_unchanged = false
|
exclude_unchanged = false
|
||||||
|
@ -3,6 +3,8 @@ dev-mode: true
|
|||||||
web:
|
web:
|
||||||
addr: :8000
|
addr: :8000
|
||||||
shutdown-timeout: 10
|
shutdown-timeout: 10
|
||||||
|
token-secret: nzMC12IJBMiiV2AAktTFpZP4BbGAf09lFPV_sATKcwI
|
||||||
|
token-expiry-time: 24h
|
||||||
|
|
||||||
db:
|
db:
|
||||||
# DB host
|
# DB host
|
||||||
|
@ -35,6 +35,7 @@ import (
|
|||||||
"git.vinchent.xyz/vinchent/howmuch/internal/howmuch/infra/router"
|
"git.vinchent.xyz/vinchent/howmuch/internal/howmuch/infra/router"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/howmuch/registry"
|
"git.vinchent.xyz/vinchent/howmuch/internal/howmuch/registry"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/log"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/log"
|
||||||
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/token"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/pkg/version/verflag"
|
"git.vinchent.xyz/vinchent/howmuch/pkg/version/verflag"
|
||||||
"github.com/gin-gonic/gin"
|
"github.com/gin-gonic/gin"
|
||||||
"github.com/jackc/pgx/v5"
|
"github.com/jackc/pgx/v5"
|
||||||
@ -125,6 +126,9 @@ func run() error {
|
|||||||
}
|
}
|
||||||
defer dbConn.Close(context.Background())
|
defer dbConn.Close(context.Background())
|
||||||
|
|
||||||
|
// Init token
|
||||||
|
token.Init(viper.GetString("web.token-secret"), viper.GetDuration("web.token-expiry-time"))
|
||||||
|
|
||||||
// Register the core service
|
// Register the core service
|
||||||
r := registry.NewRegistry(dbConn)
|
r := registry.NewRegistry(dbConn)
|
||||||
|
|
||||||
|
@ -23,6 +23,8 @@
|
|||||||
package router
|
package router
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"net/http"
|
||||||
|
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/howmuch/adapter/controller"
|
"git.vinchent.xyz/vinchent/howmuch/internal/howmuch/adapter/controller"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/core"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/core"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/errno"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/errno"
|
||||||
@ -53,6 +55,12 @@ func Routes(engine *gin.Engine, c controller.AppController) *gin.Engine {
|
|||||||
userV1 := v1.Group("/user")
|
userV1 := v1.Group("/user")
|
||||||
{
|
{
|
||||||
userV1.POST("/create", func(ctx *gin.Context) { c.User.Create(ctx) })
|
userV1.POST("/create", func(ctx *gin.Context) { c.User.Create(ctx) })
|
||||||
|
|
||||||
|
userV1.Use(middleware.Authn())
|
||||||
|
userV1.GET(
|
||||||
|
":id/info",
|
||||||
|
func(ctx *gin.Context) { ctx.JSON(http.StatusOK, "Hello world") },
|
||||||
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
v1.POST("/session/create", func(ctx *gin.Context) { c.Session.Create(ctx) })
|
v1.POST("/session/create", func(ctx *gin.Context) { c.Session.Create(ctx) })
|
||||||
|
@ -27,7 +27,7 @@ import (
|
|||||||
"sync"
|
"sync"
|
||||||
|
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/core"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/core"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/middleware"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/shared"
|
||||||
"go.uber.org/zap"
|
"go.uber.org/zap"
|
||||||
"go.uber.org/zap/zapcore"
|
"go.uber.org/zap/zapcore"
|
||||||
)
|
)
|
||||||
@ -110,8 +110,12 @@ func CtxLog(ctx core.Context) *zapLogger {
|
|||||||
func (z *zapLogger) CtxLog(ctx core.Context) *zapLogger {
|
func (z *zapLogger) CtxLog(ctx core.Context) *zapLogger {
|
||||||
zz := z.clone()
|
zz := z.clone()
|
||||||
|
|
||||||
if rid := ctx.GetHeader(middleware.XRequestID); rid != "" {
|
if rid := ctx.GetHeader(shared.XRequestID); rid != "" {
|
||||||
zz.z = zz.z.With(zap.Any(middleware.XRequestID, rid))
|
zz.z = zz.z.With(zap.Any(shared.XRequestID, rid))
|
||||||
|
}
|
||||||
|
|
||||||
|
if user := ctx.GetHeader(shared.XUserName); user != "" {
|
||||||
|
zz.z = zz.z.With(zap.Any(shared.XUserName, user))
|
||||||
}
|
}
|
||||||
|
|
||||||
return zz
|
return zz
|
||||||
|
@ -27,6 +27,7 @@ import (
|
|||||||
|
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/core"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/core"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/errno"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/errno"
|
||||||
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/shared"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/token"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/token"
|
||||||
"github.com/gin-gonic/gin"
|
"github.com/gin-gonic/gin"
|
||||||
)
|
)
|
||||||
@ -37,8 +38,6 @@ var ErrTokenInvalid = &errno.Errno{
|
|||||||
Message: "invalid token",
|
Message: "invalid token",
|
||||||
}
|
}
|
||||||
|
|
||||||
const XUserName = "X-Username"
|
|
||||||
|
|
||||||
// Authn authenticates a user's access by validating their token.
|
// Authn authenticates a user's access by validating their token.
|
||||||
func Authn() gin.HandlerFunc {
|
func Authn() gin.HandlerFunc {
|
||||||
return func(ctx *gin.Context) {
|
return func(ctx *gin.Context) {
|
||||||
@ -46,11 +45,12 @@ func Authn() gin.HandlerFunc {
|
|||||||
if err != nil || tk == nil {
|
if err != nil || tk == nil {
|
||||||
core.WriteResponse(ctx, ErrTokenInvalid, nil)
|
core.WriteResponse(ctx, ErrTokenInvalid, nil)
|
||||||
ctx.Abort()
|
ctx.Abort()
|
||||||
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: check if the key is on logout blacklist.
|
// TODO: check if the key is on logout blacklist.
|
||||||
|
|
||||||
ctx.Header(XUserName, tk.Identity)
|
ctx.Header(shared.XUserName, tk.Identity)
|
||||||
ctx.Next()
|
ctx.Next()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -23,24 +23,23 @@
|
|||||||
package middleware
|
package middleware
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/shared"
|
||||||
"github.com/gin-gonic/gin"
|
"github.com/gin-gonic/gin"
|
||||||
"github.com/google/uuid"
|
"github.com/google/uuid"
|
||||||
)
|
)
|
||||||
|
|
||||||
const XRequestID = "X-Request-Id"
|
|
||||||
|
|
||||||
func RequestID() gin.HandlerFunc {
|
func RequestID() gin.HandlerFunc {
|
||||||
return func(ctx *gin.Context) {
|
return func(ctx *gin.Context) {
|
||||||
var rid string
|
var rid string
|
||||||
|
|
||||||
if rid = ctx.GetHeader(XRequestID); rid != "" {
|
if rid = ctx.GetHeader(shared.XRequestID); rid != "" {
|
||||||
ctx.Request.Header.Add(XRequestID, rid)
|
ctx.Request.Header.Add(shared.XRequestID, rid)
|
||||||
ctx.Next()
|
ctx.Next()
|
||||||
}
|
}
|
||||||
|
|
||||||
rid = uuid.NewString()
|
rid = uuid.NewString()
|
||||||
ctx.Request.Header.Add(XRequestID, rid)
|
ctx.Request.Header.Add(shared.XRequestID, rid)
|
||||||
ctx.Header(XRequestID, rid)
|
ctx.Header(shared.XRequestID, rid)
|
||||||
ctx.Next()
|
ctx.Next()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -26,6 +26,7 @@ import (
|
|||||||
"net/http"
|
"net/http"
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/shared"
|
||||||
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/test"
|
"git.vinchent.xyz/vinchent/howmuch/internal/pkg/test"
|
||||||
"github.com/gin-gonic/gin"
|
"github.com/gin-gonic/gin"
|
||||||
"github.com/google/uuid"
|
"github.com/google/uuid"
|
||||||
@ -39,12 +40,12 @@ func TestRequestID(t *testing.T) {
|
|||||||
wanted := "123"
|
wanted := "123"
|
||||||
|
|
||||||
r.GET("/example", func(c *gin.Context) {
|
r.GET("/example", func(c *gin.Context) {
|
||||||
got = c.GetHeader(XRequestID)
|
got = c.GetHeader(shared.XRequestID)
|
||||||
c.Status(http.StatusOK)
|
c.Status(http.StatusOK)
|
||||||
})
|
})
|
||||||
|
|
||||||
r.POST("/example", func(c *gin.Context) {
|
r.POST("/example", func(c *gin.Context) {
|
||||||
got = c.GetHeader(XRequestID)
|
got = c.GetHeader(shared.XRequestID)
|
||||||
c.String(http.StatusAccepted, "ok")
|
c.String(http.StatusAccepted, "ok")
|
||||||
})
|
})
|
||||||
|
|
||||||
@ -55,12 +56,12 @@ func TestRequestID(t *testing.T) {
|
|||||||
"GET",
|
"GET",
|
||||||
"/example?a=100",
|
"/example?a=100",
|
||||||
nil,
|
nil,
|
||||||
test.Header{Key: XRequestID, Value: wanted},
|
test.Header{Key: shared.XRequestID, Value: wanted},
|
||||||
)
|
)
|
||||||
assert.Equal(t, "123", got)
|
assert.Equal(t, "123", got)
|
||||||
|
|
||||||
res := test.PerformRequest(t, r, "GET", "/example?a=100", nil)
|
res := test.PerformRequest(t, r, "GET", "/example?a=100", nil)
|
||||||
assert.NotEqual(t, "", got)
|
assert.NotEqual(t, "", got)
|
||||||
assert.NoError(t, uuid.Validate(got))
|
assert.NoError(t, uuid.Validate(got))
|
||||||
assert.Equal(t, res.Header()[XRequestID][0], got)
|
assert.Equal(t, res.Header()[shared.XRequestID][0], got)
|
||||||
}
|
}
|
||||||
|
6
internal/pkg/shared/const.go
Normal file
6
internal/pkg/shared/const.go
Normal file
@ -0,0 +1,6 @@
|
|||||||
|
package shared
|
||||||
|
|
||||||
|
const (
|
||||||
|
XRequestID = "X-Request-Id"
|
||||||
|
XUserName = "X-Username"
|
||||||
|
)
|
Loading…
Reference in New Issue
Block a user