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

Add wrapper method #478

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gustafhalldor
Copy link

Thank you for creating this handy rate limiter :)

The reason for this PR is because the industry is moving away from the word "whitelist" and replacing it with less loaded words. One of such words is "AllowedList". There are several search results on the web that explain why.

Corporate code scans are more and more frequently checking to see if the code base contains "whitelist"/"blacklist" and may reject code changes if they are found.

Would be appreciated if this one could be merged.

Thanks a bunch.

@gustafhalldor gustafhalldor changed the title Renamed property Add wrapper method Mar 25, 2024
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.

None yet

2 participants