Skip to content

Commit 336d747

Browse files
committed
自定义logger保留gorm的格式输出控制台, 修改SetUserInfo的校验规则为IdVerify
1 parent cab8de7 commit 336d747

File tree

6 files changed

+75
-24
lines changed

6 files changed

+75
-24
lines changed

server/api/v1/sys_user.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ func DeleteUser(c *gin.Context) {
240240
func SetUserInfo(c *gin.Context) {
241241
var user model.SysUser
242242
_ = c.ShouldBindJSON(&user)
243-
if err := utils.Verify(user, utils.SetUserVerify); err != nil {
243+
if err := utils.Verify(user, utils.IdVerify); err != nil {
244244
response.FailWithMessage(err.Error(), c)
245245
return
246246
}

server/config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ mysql:
6060
max-idle-conns: 10
6161
max-open-conns: 100
6262
log-mode: false
63-
log-zap: false
63+
log-zap: ""
6464

6565
# local configuration
6666
local:

server/config/gorm.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,5 @@ type Mysql struct {
99
MaxIdleConns int `mapstructure:"max-idle-conns" json:"maxIdleConns" yaml:"max-idle-conns"`
1010
MaxOpenConns int `mapstructure:"max-open-conns" json:"maxOpenConns" yaml:"max-open-conns"`
1111
LogMode bool `mapstructure:"log-mode" json:"logMode" yaml:"log-mode"`
12-
LogZap bool `mapstructure:"log-zap" json:"logZap" yaml:"log-zap"`
12+
LogZap string `mapstructure:"log-zap" json:"logZap" yaml:"log-zap"`
1313
}

server/initialize/gorm.go

Lines changed: 47 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,11 @@ import (
1010
"os"
1111
)
1212

13-
// Gorm 初始化数据库并产生数据库全局变量
13+
//@author: SliverHorn
14+
//@function: Gorm
15+
//@description: 初始化数据库并产生数据库全局变量
16+
//@return: *gorm.DB
17+
1418
func Gorm() *gorm.DB {
1519
switch global.GVA_CONFIG.System.DbType {
1620
case "mysql":
@@ -20,7 +24,12 @@ func Gorm() *gorm.DB {
2024
}
2125
}
2226

23-
// MysqlTables 注册数据库表专用
27+
// MysqlTables
28+
//@author: SliverHorn
29+
//@function: MysqlTables
30+
//@description: 注册数据库表专用
31+
//@param: db *gorm.DB
32+
2433
func MysqlTables(db *gorm.DB) {
2534
err := db.AutoMigrate(
2635
model.SysUser{},
@@ -52,7 +61,12 @@ func MysqlTables(db *gorm.DB) {
5261
global.GVA_LOG.Info("register table success")
5362
}
5463

55-
// GormMysql 初始化Mysql数据库
64+
//
65+
//@author: SliverHorn
66+
//@function: GormMysql
67+
//@description: 初始化Mysql数据库
68+
//@return: *gorm.DB
69+
5670
func GormMysql() *gorm.DB {
5771
m := global.GVA_CONFIG.Mysql
5872
dsn := m.Username + ":" + m.Password + "@tcp(" + m.Path + ")/" + m.Dbname + "?" + m.Config
@@ -76,23 +90,45 @@ func GormMysql() *gorm.DB {
7690
}
7791
}
7892

79-
// gormConfig 根据配置决定是否开启日志
93+
//@author: SliverHorn
94+
//@function: gormConfig
95+
//@description: 根据配置决定是否开启日志
96+
//@param: mod bool
97+
//@return: *gorm.Config
98+
8099
func gormConfig(mod bool) *gorm.Config {
81-
if global.GVA_CONFIG.Mysql.LogZap {
100+
switch global.GVA_CONFIG.Mysql.LogZap {
101+
case "Silent":
82102
return &gorm.Config{
83-
Logger: Default.LogMode(logger.Info),
103+
Logger: Default.LogMode(logger.Silent),
84104
DisableForeignKeyConstraintWhenMigrating: true,
85105
}
86-
}
87-
if mod {
106+
case "Error":
88107
return &gorm.Config{
89-
Logger: logger.Default.LogMode(logger.Info),
108+
Logger: Default.LogMode(logger.Error),
90109
DisableForeignKeyConstraintWhenMigrating: true,
91110
}
92-
} else {
111+
case "Warn":
93112
return &gorm.Config{
94-
Logger: logger.Default.LogMode(logger.Silent),
113+
Logger: Default.LogMode(logger.Warn),
95114
DisableForeignKeyConstraintWhenMigrating: true,
96115
}
116+
case "Info":
117+
return &gorm.Config{
118+
Logger: Default.LogMode(logger.Info),
119+
DisableForeignKeyConstraintWhenMigrating: true,
120+
}
121+
default:
122+
if mod {
123+
return &gorm.Config{
124+
Logger: logger.Default.LogMode(logger.Info),
125+
DisableForeignKeyConstraintWhenMigrating: true,
126+
}
127+
} else {
128+
return &gorm.Config{
129+
Logger: logger.Default.LogMode(logger.Silent),
130+
DisableForeignKeyConstraintWhenMigrating: true,
131+
}
132+
}
97133
}
98134
}

server/initialize/logger.go

Lines changed: 25 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ import (
1414
)
1515

1616
var (
17-
Discard = New(log.New(ioutil.Discard, "", log.LstdFlags), GormConfig{})
18-
Default = New(log.New(os.Stdout, "\r\n", log.LstdFlags), GormConfig{
17+
Discard = New(log.New(ioutil.Discard, "", log.LstdFlags), log.New(ioutil.Discard, "", log.LstdFlags), GormConfig{})
18+
Default = New(log.New(os.Stdout, "\r\n", log.LstdFlags), log.New(os.Stdout, "\r\n", log.LstdFlags), GormConfig{
1919
SlowThreshold: 200 * time.Millisecond,
2020
LogLevel: logger.Warn,
2121
Colorful: true,
@@ -31,7 +31,7 @@ type traceRecorder struct {
3131
Err error
3232
}
3333

34-
func New(writer Writer, config GormConfig) logger.Interface {
34+
func New(writer Writer, gormWriter logger.Writer, config GormConfig) logger.Interface {
3535
var (
3636
infoStr = "%s\n[info] "
3737
warnStr = "%s\n[warn] "
@@ -52,6 +52,7 @@ func New(writer Writer, config GormConfig) logger.Interface {
5252

5353
return &GormLogger{
5454
Writer: writer,
55+
gormWriter: gormWriter,
5556
GormConfig: config,
5657
infoStr: infoStr,
5758
warnStr: warnStr,
@@ -75,6 +76,7 @@ type GormConfig struct {
7576

7677
type GormLogger struct {
7778
Writer
79+
gormWriter logger.Writer
7880
GormConfig
7981
infoStr, warnStr, errStr string
8082
traceStr, traceErrStr, traceWarnStr string
@@ -135,16 +137,31 @@ func (g *GormLogger) Trace(ctx context.Context, begin time.Time, fc func() (stri
135137
}
136138

137139
func (g *GormLogger) Printf(message string, data ...interface{}) {
140+
if global.GVA_CONFIG.Mysql.LogZap == "Info" && !global.GVA_CONFIG.Mysql.LogMode {
141+
switch len(data) {
142+
case 0:
143+
global.GVA_LOG.Info(message)
144+
case 1:
145+
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]))
146+
case 2:
147+
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]), zap.Any("duration", data[1]))
148+
case 3:
149+
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]), zap.Any("duration", data[1]), zap.Any("rows", data[2]))
150+
case 4:
151+
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]), zap.Any("duration", data[1]), zap.Any("rows", data[2]), zap.Any("sql", data[3]))
152+
}
153+
return
154+
}
138155
switch len(data) {
139156
case 0:
140-
global.GVA_LOG.Info(message)
157+
g.gormWriter.Printf(message, "")
141158
case 1:
142-
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]))
159+
g.gormWriter.Printf(message, data[0].(string))
143160
case 2:
144-
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]), zap.Any("duration", data[1]))
161+
g.gormWriter.Printf(message, data[0].(string), data[1].(float64))
145162
case 3:
146-
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]), zap.Any("duration", data[1]), zap.Any("rows", data[2]))
163+
g.gormWriter.Printf(message, data[0].(string), data[1].(float64), data[2].(string))
147164
case 4:
148-
global.GVA_LOG.Info("gorm", zap.Any("src", data[0]), zap.Any("duration", data[1]), zap.Any("rows", data[2]), zap.Any("sql", data[3]))
165+
g.gormWriter.Printf(message, data[0].(string), data[1].(float64), data[2].(string), data[3].(string))
149166
}
150167
}

server/utils/verify.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,10 @@ var (
66
MenuVerify = Rules{"Path": {NotEmpty()}, "ParentId": {NotEmpty()}, "Name": {NotEmpty()}, "Component": {NotEmpty()}, "Sort": {Ge("0")}}
77
MenuMetaVerify = Rules{"Title": {NotEmpty()}}
88
LoginVerify = Rules{"CaptchaId": {NotEmpty()}, "Captcha": {NotEmpty()}, "Username": {NotEmpty()}, "Password": {NotEmpty()}}
9-
SetUserVerify = Rules{"ID": {NotEmpty()}, "Username": {NotEmpty()}, "NickName": {NotEmpty()}, "HeaderImg": {NotEmpty()}}
109
RegisterVerify = Rules{"Username": {NotEmpty()}, "NickName": {NotEmpty()}, "Password": {NotEmpty()}, "AuthorityId": {NotEmpty()}}
1110
PageInfoVerify = Rules{"Page": {NotEmpty()}, "PageSize": {NotEmpty()}}
1211
CustomerVerify = Rules{"CustomerName": {NotEmpty()}, "CustomerPhoneData": {NotEmpty()}}
1312
AutoCodeVerify = Rules{"Abbreviation": {NotEmpty()}, "StructName": {NotEmpty()}, "PackageName": {NotEmpty()}, "Fields": {NotEmpty()}}
14-
WorkFlowVerify = Rules{"WorkflowNickName": {NotEmpty()}, "WorkflowName": {NotEmpty()}, "WorkflowDescription": {NotEmpty()}, "WorkflowStepInfo": {NotEmpty()}}
1513
AuthorityVerify = Rules{"AuthorityId": {NotEmpty()}, "AuthorityName": {NotEmpty()}, "ParentId": {NotEmpty()}}
1614
AuthorityIdVerify = Rules{"AuthorityId": {NotEmpty()}}
1715
OldAuthorityVerify = Rules{"OldAuthorityId": {NotEmpty()}}

0 commit comments

Comments
 (0)