envoyproxy / envoyproxy/gateway

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

Open
#5,256 6 comments 2 reactions 0 assignees View on GitHub
help wanted kind/bug
Dominant language
Go
Stars
3k
Forks
864
Avg merge
2d 2h
Merged PRs (30d)
140

Description

*Description*:

In its [examples](https://gateway-api.sigs.k8s.io/reference/spec/#gateway.networking.k8s.io%2fv1.HTTPPathModifier), 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

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.