envoyproxy/gateway

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

Open

#5.256 geöffnet am 11. Feb. 2025

Auf GitHub ansehen
 (6 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Go (802 Forks)auto 404
help wantedkind/bug

Repository-Metriken

Stars
 (2.871 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfe den in PR #5254 hinzugefügten End to End Test und verfolge den ReplacePrefixMatch Handler des URLRewrite Filters, um zu verstehen, warum er für leere Match Präfix Pfade fehlschlägt.
Tech Stack
go
Domain
api
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65