Skip to content

Don't warn in setRateLimitGroup for bypassed requests - #394

Merged
hansott merged 3 commits into
mainfrom
bypassed-context-flag
Oct 20, 2026
Merged

hansott merged 3 commits into
mainfrom
bypassed-context-flag

Conversation

@hansott

@hansott hansott commented Oct 20, 2026

Copy link
Copy Markdown
Member

No description provided.

threadLocalBypassed.set(true);
}
public static boolean isBypassed() {
return threadLocalBypassed.get() != null;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So a false value will still return true?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@hansott
hansott force-pushed the bypassed-context-flag branch from f5aa5ba to 67a0724 Compare October 20, 2026 10:55
@codecov

codecov Bot commented Oct 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@hansott
hansott merged commit dc5a45f into main Oct 20, 2026
283 checks passed
@hansott
hansott deleted the bypassed-context-flag branch October 20, 2026 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants