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

♻️ replace deprecated per_second with seconds_per_request function #616

Merged

Conversation

jurijsb
Copy link
Contributor

@jurijsb jurijsb commented Oct 6, 2024

What does this PR do?

Updates rate-limiting middleware to use the new seconds_per_request function and removes usage of the deprecated per_second function.

Why is this change important?

Using deprecated functions can lead to compatibility issues with future updates of the middleware or framework. By adopting seconds_per_request, you're ensuring that your code remains compatible with upcoming versions, reducing the risk of breaking changes in the future.

Related issues

Closes #614

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congrats on making your first Pull Request and thanks for taking the time to improve Websurfx! ❤️!
Say hello by joining the conversation in our Discord

@jurijsb jurijsb marked this pull request as ready for review October 6, 2024 14:58
@neon-mmd neon-mmd changed the title refactor: replace deprecated per_second with seconds_per_request ♻️ replace deprecated per_second with seconds_per_request function Oct 6, 2024
Copy link
Owner

@neon-mmd neon-mmd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks ❤️ for working on such a wonderful PR. We really appreciate it. Good job!! Keep it up!! 👍 Since everything looks good. We will merge this PR right away 🚀 🙂 .

@neon-mmd neon-mmd merged commit 959d0c5 into neon-mmd:rolling Oct 6, 2024
5 checks passed
@jurijsb jurijsb deleted the issue-614-replace-function-per_second branch October 6, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

👽️ Replace the function per_second with seconds_per_request for rate-limiting middleware
2 participants