New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: add parameter verification to the visit counter API #3546
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3546 +/- ##
============================================
- Coverage 59.77% 59.61% -0.16%
Complexity 2189 2189
============================================
Files 337 337
Lines 11273 11305 +32
Branches 837 844 +7
============================================
+ Hits 6738 6740 +2
- Misses 4129 4159 +30
Partials 406 406
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
需要先解决一下以下冲突:
|
/ping @halo-dev/sig-halo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnNiang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
What type of PR is this?
/kind improvement
/area core
/milestone 2.4.x
What this PR does / why we need it:
对访问量统计的 API 增加数据合法性校验
点赞同理
see #3530 for more details
how to test it?
期望出现日志:
并且
不会出现上述错误数据的记录
Which issue(s) this PR fixes:
Fixes #3530
Does this PR introduce a user-facing change?