Skip to content

tiered rate limit jwt. e2e tests #7411

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
Closed
pdabelf5 wants to merge 23 commits into from
Closed

tiered rate limit jwt. e2e tests #7411

pdabelf5 wants to merge 23 commits into from

Conversation

@pdabelf5
Copy link
Collaborator

@pdabelf5 pdabelf5 commented Feb 26, 2025

Proposed changes

  • added fix to virtualserver.go to replace all hyphens in nginx variables to underscores

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

haywoodsh and others added 23 commits February 20, 2025 15:50
@haywoodsh @pdabelf5
generate auth_jwt_claim_set directive
0d050bc 
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>

Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
@haywoodsh @pdabelf5
add auth_jwt_claim_set to spec, route, subroute
5f9b5d6 
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>

Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
@haywoodsh @pdabelf5
add test, fix template, remove pointers, generate crd
7559f3f 
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>

Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
@pdabelf5
move AuthJWTClaimSets to policyCfg.RateLimit
851abfe
@pdabelf5
generate maps for tiered rate limit groups
2404724
@nginx-bot @pdabelf5
Docker image update bc2e599d (#7256)
e16e8db 
Update docker images bc2e599d

Co-authored-by: Paul Abel <128620221+pdabelf5@users.noreply.github.com>
@pdabelf5
format ratelimit zone key
6ddd623
@pdabelf5
add rate limit claim group maps
a0012db
@pdabelf5
Merge branch 'main' into feat/rate-limit-group-maps-v2
fe4011d
@pdabelf5
allow container image to be built in project root or tests directory
116ad48
@pdabelf5
add vsr test to rate limit by JWT claim
86a0c29
@pdabelf5
make all nginx variables use underscores rather than hyphens
23cebf2
@pdabelf5
add e2e tests for tiered rate limits
eaba087
@pdabelf5
make apply_and_assert_valid_policy() policy function
5a53703
@pdabelf5
make check_rate_limit() for re-usable rate limit check logic
24c5f4e
@pdabelf5
refactor existing tests to use check_rate_limit()
3a7551e
@pdabelf5
update to use vsr patch and assert function
83e1a96
@pdabelf5
correct variable name
950aced
@pdabelf5
update and assert vs in rate limit policies
1f9bbe1
@pdabelf5
allow to suppress config print
5ea7eb9
@pdabelf5
allow all tests in the class to run and complete
19e0e3b
@pdabelf5
add test where vsr & vs get different tiered rate limits
7b0a075
@pdabelf5
add tests to check for duplicate default jwt conditions
72f123e
@github-actions github-actions bot added go Pull requests that update Go code python Pull requests that update Python code docker Pull requests that update Docker code labels Feb 26, 2025
@pdabelf5 pdabelf5 closed this Feb 26, 2025
@pdabelf5 pdabelf5 deleted the test/tiered-rate-limit-jwt branch February 26, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

No one assigned

Labels

docker Pull requests that update Docker code go Pull requests that update Go code python Pull requests that update Python code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pdabelf5 @haywoodsh @nginx-bot