jetty/jetty.project

Why there is no Regex Support for excludeList of nonProxyHost

Aperta

#8329 aperta il 22 lug 2022

Vedi su GitHubGuida contributor
 (10 commenti) (0 reazioni) (1 assegnatario)Java (1913 fork)batch import
EnhancementHelp Wanted

Metriche repository

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

Descrizione

Code at line below should be enhanced: https://github.com/eclipse/jetty.project/blob/8102378e2d24b8117ed3a545c7579f79d35fc45a/jetty-client/src/main/java/org/eclipse/jetty/client/ProxyConfiguration.java#L180

Generally nonProxyHosts gets defined with wildcards like *.abc.com|*.svc.local etc.. so .equals doesn't help ! And apparently Jetty Client doesn't have any support for wildcard list of nonProxyHosts !!

PS: reactor-netty handles it pretty well, refer below: https://github.com/reactor/reactor-netty/blob/main/reactor-netty-core/src/main/java/reactor/netty/transport/ProxyProvider.java#L310

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente di ProxyConfiguration.java, comprendi come vengono confrontati i nonProxyHosts e aggiungi un meccanismo di corrispondenza basato su regex simile a ProxyProvider di reactor netty.
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-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
JavaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45