spring-cloud/spring-cloud-gateway

Make RedisRateLimiter extensible?

Aperta

#393 aperta il 26 giu 2018

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)Java (3204 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (4284 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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!

Guida contributor

Direzione di ricerca
Studia la struttura attuale della classe RedisRateLimiter, comprendi i suoi metodi privati e con ambito predefinito, e identifica quali parti devono essere rese protected o public per consentire l'estensione. Rivedi anche la configurazione del limite di velocità di Spring Cloud Gateway per capire come viene creato il bean e come un'implementazione personalizzata può sostituirlo.
Tech stack
java
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaSpringRedisGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50