增加 csrf 的判断 #55
Open
增加 csrf 的判断 #55
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
polaris1119
reviewed
Feb 26, 2018
| DIR=`pwd` | ||
| export GOPATH= | ||
| export GOPATH=$DIR | ||
| export PATH=$PATH:$DIR/bin |
There was a problem hiding this comment.
这个文件是干嘛用的？将 studygolang 执行文件加入 PATH ?
| @@ -19,3 +19,6 @@ stop: | |||
|
|
|||
| migrate: | |||
| ./bin/migrator --changeVersion=${v} | |||
|
|
|||
| run-studygolang: | |||
| cd src/server/studygolang; go run `ls | grep -v windows`; cd - | |||
There was a problem hiding this comment.
这里的用途是？
结合 env.sh 然后 执行 make run-studygolang ? 感觉是不是没啥必要？
There was a problem hiding this comment.
start做的事情是需要先 build ，然后在start。如果每次都写一点东西然后build，然后再start感觉有点效率低下。
| @@ -370,6 +370,8 @@ func (UserLogic) Total() int64 { | |||
| var ( | |||
| ErrUsername = errors.New("用户名不存在") | |||
| ErrPasswd = errors.New("密码错误") | |||
|
|
|||
| ErrUnameOrPwd = errors.New("用户名或密码错误") | |||
polaris1119
reviewed
Feb 27, 2018
| @@ -62,6 +62,7 @@ func main() { | |||
| serveStatic(e) | |||
|
|
|||
| e.Use(thirdmw.EchoLogger()) | |||
| e.Use(pwm.CsrfRefererFilter()) | |||
There was a problem hiding this comment.
项目中还有一个 AppGroup，也就是用于 APP 接口的，而接口没有 csrf 一说。所以，这个中间件，加到 frontG 和 adminG 上面吧。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.