Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: demote rate limit log level to notice #9134

Merged
merged 1 commit into from
Oct 12, 2022
Merged

fix: demote rate limit log level to notice #9134

merged 1 commit into from
Oct 12, 2022

Conversation

id
Copy link
Collaborator

@id id commented Oct 11, 2022 •
edited
Loading

fixes #9096

@id id force-pushed the fix-demote-rate-limit-log-level-to-notice branch from 38e4d5f to 73d72ea Compare October 11, 2022 11:05
@qzhuyan qzhuyan requested a review from lafirest October 11, 2022 11:55
qzhuyan
qzhuyan approved these changes Oct 11, 2022
View reviewed changes
Copy link
Contributor

@qzhuyan qzhuyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qzhuyan
Copy link
Contributor

qzhuyan commented Oct 11, 2022

wait for approval from @lafirest

@id id merged commit 18db788 into main-v4.3 Oct 12, 2022
@id id deleted the fix-demote-rate-limit-log-level-to-notice branch October 12, 2022 08:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@qzhuyan qzhuyan qzhuyan approved these changes

@HJianBo HJianBo HJianBo approved these changes

@lafirest lafirest lafirest approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@id @qzhuyan @HJianBo @lafirest