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

Move rate-limiting from per-client to per-client-per-domain #1468

Closed
wants to merge 3 commits into from
Closed

Move rate-limiting from per-client to per-client-per-domain #1468

wants to merge 3 commits into from

Commits on Nov 21, 2022

  1. Move rate-limiting from per-client to per-client-per-domain 

    Signed-off-by: DL6ER <dl6er@dl6er.de>
    @DL6ER
    DL6ER committed Nov 21, 2022
    Configuration menu
    Copy the full SHA
    00adb82 View commit details
    Browse the repository at this point in the history

  2. Get message table type string representations from a getter instead o… 

    …f an array of strings to ensure we cannot forget to update the strings when we add new types in the future (the compiler would complain about not having handled all possible switch cases)

Signed-off-by: DL6ER <dl6er@dl6er.de>
    @DL6ER
    DL6ER committed Nov 21, 2022
    Configuration menu
    Copy the full SHA
    b17bdff View commit details
    Browse the repository at this point in the history

  3. Lower default rate-limiting values as we are not selective not only p… 

    …er-client but also per-domain

Signed-off-by: DL6ER <dl6er@dl6er.de>
    @DL6ER
    DL6ER committed Nov 21, 2022
    Configuration menu
    Copy the full SHA
    d551572 View commit details
    Browse the repository at this point in the history