spring-cloud/spring-cloud-gateway

Make RedisRateLimiter extensible?

Ouverte

#393 ouverte le 26 juin 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (1 réaction) (0 personne assignée)Java (3 204 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (4 284 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Hey guys!

In order to be able to rate limit per user API key (versus basing the limit on the route) I had to extend RedisRateLimiter to make isAllowed(String routeId, String apiKey) calculate the limits based up on the route and the apiKey. Please see the full thing here: https://stackoverflow.com/questions/50988095/rate-limiting-based-on-user-plan-in-spring-cloud-gateway

As you see it is somewhat hackish there because there's no good way to extend RedisRateLimiter - see I had to decorate it, copy some private/default scoped methods over as well as hack the @Primary annotation so it gets into the bean factory well.

Do you think RedisRateLimiter would be re-architected to be easier to extend? Thanks!

Guide contributeur

Direction de recherche
Étudiez la structure actuelle de la classe RedisRateLimiter, comprenez ses méthodes privées et celles avec la portée par défaut, et identifiez les parties qui doivent être rendues protected ou public pour permettre l'extension. Examinez également la configuration de limitation de débit de Spring Cloud Gateway pour comprendre comment le bean est créé et comment une implémentation personnalisée peut le remplacer.
Stack technique
java
Domaine
backendapi
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaSpringRedisGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50