alibaba/Sentinel

一点关于UrlCleaner的改进建议和意见

Open

#1287 aperta il 14 feb 2020

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Java (8150 fork)batch import
area/integrationsgood first issuekind/enhancement

Metriche repository

Star
 (23.109 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

这里，UrlCleaner特指sentinel-spring-webmvc-adapter中的UrlCleaner： https://github.com/alibaba/Sentinel/blob/master/sentinel-adapter/sentinel-spring-webmvc-adapter/src/main/java/com/alibaba/csp/sentinel/adapter/spring/webmvc/callback/UrlCleaner.java

个人认为有两点值得改进：

  • UrlCleaner名称词不达意。UrlCleaner重写的不是Url，而是资源名称，叫ResourceNameRewiter或许更加贴切。

  • 目前UrlCleaner提供的参数较少，能定制的空间很小。目前只能在原资源名称的基础上改写资源。但实际业务中，往往需要根据具体的请求重写资源名称。因此，加上 HttpServletRequest 的参数灵活性会好很多。

Guida contributor

Direzione di ricerca
Rinominare UrlCleaner in ResourceNameRewriter e aggiungere il parametro HttpServletRequest per consentire la riscrittura personalizzata in base ai dettagli della richiesta.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Java
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70