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

feat(jans-auth-server): introduce rate limit support for DCR #10931

Open
3 tasks
yuriyz opened this issue Feb 23, 2025 · 1 comment
Open
3 tasks

feat(jans-auth-server): introduce rate limit support for DCR #10931

yuriyz opened this issue Feb 23, 2025 · 1 comment
Assignees
@yuriyz
Labels
comp-jans-auth-server Component affected by issue or PR kind-feature Issue or PR is a new feature request

Comments

@yuriyz
Copy link
Contributor

yuriyz commented Feb 23, 2025

Description

add rate limit support for DCR. It will work based on SSA. E.g. same SSA can be used not more then 5 times per minute. Otherwise call will be rejected with http status code 429 (too many requests).

Test cases and code coverage

  • Write unit test to cover added/changed code
  • Update integration tests to cover added/changed code
  • documentation
@yuriyz yuriyz self-assigned this Feb 23, 2025
@mo-auto mo-auto added comp-jans-auth-server Component affected by issue or PR kind-feature Issue or PR is a new feature request labels Feb 23, 2025
@yuriyz
Copy link
Contributor Author

yuriyz commented Feb 24, 2025

We need to rate limit also redirect_uri.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuriyz yuriyz

Labels
comp-jans-auth-server Component affected by issue or PR kind-feature Issue or PR is a new feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yuriyz @mo-auto