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.

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

Wildcard config support for routes / methods #115

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Conversation

tudor-timcu
Copy link
Collaborator

@tudor-timcu tudor-timcu commented Dec 20, 2024

  • Added support for configs that use wildcards when adding routes / method in the Zen dashboard
  • Updated rate limiting logic to also maintain counts for the wildcard endpoints (if they match) and provide the rate limiting status back to the request process if threshold is exceeded
  • Added support for selecting the most restrictive config from multiple matching ones:
    • If at least one force protection off is set to true in all the matching endpoints configs -> the protection is of
    • The allowedIPAddresses fields are checked for all the matching routes and if it's found in at least one -> the rest of the checks are bypassed
    • Same for checking if the rate limiting is enabled for the current route, if at least one config has rate limiting enabled for the matching endpoints -> we do the gRPC request to get the rate limiting status from agent
  • Added E2E tests to check these config selection cases + rate limiting on wildcard routes

@tudor-timcu tudor-timcu changed the title [WIP] Wildcard support for routes / methods Wildcard config support for routes / methods Jan 30, 2025
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.

3 participants