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

Refactor the ratelimiting behavior out of the request handlers. #226

Merged
merged 4 commits into from
Mar 11, 2024
Merged

Refactor the ratelimiting behavior out of the request handlers. #226

merged 4 commits into from
Mar 11, 2024

Commits on Mar 10, 2024

  1. introduce a ratelimiting decorator 

    The ratelimit_endpoint decorator will wrap any request handler's function to handle the ratelimit for it.

This avoids having the ratelimit checks being done in the handler's callback function's body
    @shtlrs
    shtlrs committed Mar 10, 2024
    Configuration menu
    Copy the full SHA
    4d24d75 View commit details
    Browse the repository at this point in the history

  2. move the error handling logic to write_error for the /curl handler

    @shtlrs
    shtlrs committed Mar 10, 2024
    Configuration menu
    Copy the full SHA
    84f27d0 View commit details
    Browse the repository at this point in the history

  3. decorate all request handler's callback functions with the ratelimiter

    @shtlrs
    shtlrs committed Mar 10, 2024
    Configuration menu
    Copy the full SHA
    57cbca4 View commit details
    Browse the repository at this point in the history

  4. rename the ratelimiter functions 

    This is to improve the decorator's api better.
    @shtlrs
    shtlrs committed Mar 10, 2024
    Configuration menu
    Copy the full SHA
    979d47a View commit details
    Browse the repository at this point in the history