[WIP]jwt design #85
Closed
[WIP]jwt design #85
Conversation
This file contains 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
|
同步一下最新的go.mod，学一下基本git用法 |
ozline
reviewed
Nov 8, 2024
api/middleware/rate_limit.go
Outdated
| limiter, exists := limiterMap[clientIP] | ||
| if !exists { | ||
| // 创建一个新的速率限制器，限制每秒rps次请求，允许burst次突发请求 | ||
| limiter = rate.NewLimiter(rate.Limit(rps), burst) |
There was a problem hiding this comment.
确实是这样，我貌似把issue里的需求理解错了，是为了防止第三方恶意盗用接口，防刷的话sentinal确实已经解决了，这个中间件就不需要了。
api/middleware/jwt.go
Outdated
| } | ||
| username := loginStruct.Username | ||
| password := loginStruct.Password | ||
| if !(username == "admin" && password == "admin") { |
api/middleware/jwt.go
Outdated
| TokenHeadName: "Bearer", | ||
| // 登录成功后的响应 | ||
| LoginResponse: func(ctx context.Context, c *app.RequestContext, code int, token string, expire time.Time) { | ||
| c.JSON(http.StatusOK, utils.H{ |
api/middleware/jwt.go
Outdated
| func InitJwt() { | ||
| var err error | ||
| JwtMiddleware, err = jwt.New(&jwt.HertzJWTMiddleware{ | ||
| Realm: "test zone", |
SmilingSea
force-pushed
the
jwt_design
branch
from
1e92d3b
to
7548963
Compare
Codecov ReportAttention: Patch coverage is
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
有一个疑问就是 我在签发token的时候 得到传入的参数之后（id password这些）要去哪里验证 |
|
GitHub 账户学一下 GPG commit verification，把前面的 git 记录删掉，现在 merge 必须要求满 verification |
SmilingSea
force-pushed
the
jwt_design
branch
from
4f2d823
to
29e7efe
Compare
|
改了一下pr 到 #99 |
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.
目前是使用的hertz提供的jwt实现github.com/hertz-contrib/jwt以及Golang提供的限流器golang.org/x/time/rate进行接口防刷实现，目前参数都还是写死的，还没有写到接口里面去，等我本地测测再pull request