Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Added audit_only, fault_tolerant and custom error_code config #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added audit_only, fault_tolerant and custom error_code config #21

wants to merge 1 commit into from

Conversation

nileshbhadana
Copy link

@nileshbhadana nileshbhadana commented Jan 11, 2024 •
edited

Summary

  • Added audit_only flag to log all the rate-limited requests while letting the requests pass through
  • Added fault_tolerant flag that determines if the requests should be proxied even if Kong has troubles connecting a redis. This was recently added to OSS version, adding it here also.
  • Added error_code config option to define custom error_code to return for rate-limited requests

Issues resolved

Fix #20

@nileshbhadana
Added audit_only, fault_tolerant flags and option to define custom st…
1000fe8 
…atus_code

Signed-off-by: Nilesh Bhadana <nileshbhadana3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anuj147 anuj147 Awaiting requested review from anuj147 anuj147 is a code owner

@chirag-manwani chirag-manwani Awaiting requested review from chirag-manwani chirag-manwani is a code owner

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.

Log only mode and Audit Trail
1 participant
@nileshbhadana