feat: init server project #5
Conversation
pull-request-size
bot
added
the
size/XXL
Whoa! Easy there, Partner!This PR is too big. Please break it up into smaller PRs. |
| Secure: false, | ||
| }, | ||
| ) | ||
| if err != nil { |
There was a problem hiding this comment.
| if err != nil { | |
| if err != nil { | |
| log.Panicf("Minio", err.Error(), " Open Bucket failed ", endpoint) | |
| } |
If an error occurs and the code simply logs the error message using the log.Panicf() function and exits, it is not friendly enough to consider returning the error message to the caller and letting the caller decide how to handle the error.
| }, | ||
| } | ||
| } | ||
|
|
There was a problem hiding this comment.
whether or not
type Config struct {
App AppConfig
JWT JWTConfig
Server ServerConfig
Mysql MysqlConfig
Redis RedisConfig
Minio MinioConfig
}
type AppConfig struct {
Version string `mapstructure:"version"`
Debug bool `mapstructure:"debug"`
LogFile string `mapstructure:"log_file"`
}viper library to read configuration files
| viper.SetConfigFile(configPath) | ||
|
|
||
| if err := viper.ReadInConfig(); err != nil { | ||
| panic(fmt.Errorf("fatal error config file: %s", err)) |
| ) | ||
|
|
||
| func init() { | ||
| configPath := "./config.yaml" |
There was a problem hiding this comment.
I need to specify a path to read the configuration file, do I need to consider reading the configuration file from the command line?
| config.ConfigInit() | ||
|
|
||
| // logo | ||
| _logo := ` |
There was a problem hiding this comment.
There was a problem hiding this comment.
Sounds good, I will try to choose one.
| // Licensed under the MIT License (the "License"); | ||
| // you may not use this file except in compliance with the License. | ||
|
|
||
| package log |
There was a problem hiding this comment.
I suggest implementing an authentication mechanism in Redis to enhance the security of its connections.
There was a problem hiding this comment.
Ok, I will update it.
| "context" | ||
| "fmt" | ||
| "io" | ||
|
|
There was a problem hiding this comment.
there is one more spacce here
🔍 What type of PR is this?
/kind feature
👀 What this PR does / why we need it:
init server project
🅰 Which issue(s) this PR fixes:
issue: #3
Fixes # Add server base project.
📝 Special notes for your reviewer:
🎯 Describe how to verify it
📑 Additional documentation e.g., RFC, notion, Google docs, usage docs, etc.: