envoyproxy/gateway

ReplacePrefixMatch in the URLRewrite filter does not handle empty match prefix correctly

Open

#5 256 ouverte le 11 févr. 2025

Voir sur GitHub
 (6 commentaires) (1 réaction) (0 assignés)Go (802 forks)auto 404
help wantedkind/bug

Métriques du dépôt

Stars
 (2 871 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Description:

In its examples, the Gateway API uses "foo" in "Prefix Match". However, we can conclude that the HTTPPathModifier should also work on / as the path prefix match.

The PR #5254 PR demonstrates that / is not handled correctly.

Repro steps:

Follow the end-2-end test, which is added in #5254

Environment:

In our environment, we use docker.io/envoyproxy/gateway:v1.3.0. The bug is reproducible in the main branch, and is reproducible in v1.2.5

Guide contributeur

Direction de recherche
Inspectez le test de bout en bout ajouté dans la PR #5254 et suivez le gestionnaire ReplacePrefixMatch du filtre URLRewrite pour comprendre pourquoi il échoue pour le chemin de préfixe de correspondance vide.
Stack technique
go
Domaine
api
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65