Alex
Alex
The issue lies in the BaseHasAPIKey permission class, particularly in the has_permission method where an API key is validated. In high-traffic applications, this validation process can be CPU-intensive, especially considering...
@florimondmanca Thank you for considering the proposed enhancement and your willingness to review PRs related to this issue. Regarding your questions: 1. If caching is not set up already: Django...
@florimondmanca Hey, Got it! I'll implement the described solution and once it's ready, I'll provide the final version for your review. Let's get this done!
@florimondmanca [Implement API key caching and update documentation](https://github.com/florimondmanca/djangorestframework-api-key/pull/242) Pull request summary: 1. Dynamic Caching Mechanism: Introduced the CacheMixin, enabling dynamic key generation based on the class name. This facilitates model/permissions...
Hi @beda42, thank you for your input on this issue. **Backward Compatibility:** Switching the hashing algorithm is an interesting suggestion but it introduces a backward compatibility issue. Existing applications using...
Hi again, I appreciate the thoughtful discussion. To strike a balance between performance and security, what if we introduce a configurable setting for the hashing algorithm and caching? This would...
@beda42 Thank you for your detailed response. Your points are valid and worthy of consideration. Let's go through them: **Backward Compatibility & Hashing Algorithm:** Having a configurable hashing algorithm could...
@florimondmanca and @beda42, 1. If the existing codebase makes use of Django's built-in password management utilities, it should already have a built-in mechanism for upgrading hash algorithms when the configured...
Hey @florimondmanca, I've submitted a new commit for pull request that introduces optional settings for caching. I believe this feature could offer more flexibility for users who want to make...