envoyproxy / envoyproxy/gateway

replacePrefixMatch broken in main

未关闭
#4,331 19 条评论 7 个 reaction 已指派 0 人 在 GitHub 查看
triage
主要语言
Go
星标
3k
派生
864
平均合并
2 天 2 小时
30 天内合并 PR
140

描述

*Description*:
>What issue is being seen? Describe what should be happening instead of
the bug, for example: Envoy should not crash, the expected value isn't
returned, etc.

with envoy gateway 1.1 I was using following spec

```
- type: URLRewrite
urlRewrite:
path:
type: ReplacePrefixMatch
replacePrefixMatch: /docs/
```

So my goal is to add `/docs/` prefix to all url paths. So it will be modified like `/docs/`.

With latest main the request path was modified only `/docs` (NOTE! missing last slash)

However, with latest main it will not add the ending slash anymore. I was forced to use

```
- type: URLRewrite
urlRewrite:
path:
type: ReplacePrefixMatch
replacePrefixMatch: /docs//
```

to get same behaviour working.

*Repro steps*:
> Include sample requests, environment, etc. All data and inputs
required to reproduce the bug.

>**Note**: If there are privacy concerns, sanitize the data prior to
sharing.

*Environment*:
>Include the environment like gateway version, envoy version and so on.

eg 1.1 (envoy 1.31) and main (with envoy distroless-devel)

*Logs*:
>Include the access logs and the Envoy logs.

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。