jetty/jetty.project

Why there is no Regex Support for excludeList of nonProxyHost

Open

#8 329 ouverte le 22 juil. 2022

Voir sur GitHub
 (10 commentaires) (0 réactions) (1 assigné)Java (1 913 forks)batch import
EnhancementHelp Wanted

Métriques du dépôt

Stars
 (3 701 stars)
Métriques de merge PR
 (Merge moyen 6j 14h) (48 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de ProxyConfiguration.java, comprenez comment les nonProxyHosts sont mis en correspondance et ajoutez un mécanisme de correspondance basé sur les regex similaire à ProxyProvider de reactor netty.
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-3 heures
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.
Claire
Prérequis
JavaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45