Skip to content

feat: Add path prefix upstream - #361

Open
yuvaldekel wants to merge 8 commits into
prometheus-community:mainfrom
yuvaldekel:add-path-prefix-upstream
Open

feat: Add path prefix upstream#361
yuvaldekel wants to merge 8 commits into
prometheus-community:mainfrom
yuvaldekel:add-path-prefix-upstream

Conversation

@yuvaldekel

Copy link
Copy Markdown
Contributor

This PR enables proxying to an upstream service with a path prefix. It introduces the --upstream-path-prefix flag, which appends a specified path to all forwarded requests.

Resolves #354

Signed-off-by: yuvaldekel <yuvddd05@gmail.com>
Signed-off-by: yuvaldekel <yuvddd05@gmail.com>
Signed-off-by: yuvaldekel <yuvddd05@gmail.com>
Signed-off-by: yuvaldekel <yuvddd05@gmail.com>
Signed-off-by: yuvaldekel <yuvddd05@gmail.com>
@simonpasquier

Copy link
Copy Markdown
Contributor

I'm not sure that we need this. Using -upstream=http://example.com/some-prefix should work IMHO.

@simonpasquier

Copy link
Copy Markdown
Contributor

Commented on #354 but my position is that we need first to understand why it doesn't work today. Implementing #336 would help.

Signed-off-by: Yuval dekel <153660450+yuvaldekel@users.noreply.github.com>
@SuperQ

SuperQ commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

This needs a rebase

yuvaldekel and others added 2 commits July 31, 2026 15:14
Signed-off-by: Yuval dekel <153660450+yuvaldekel@users.noreply.github.com>
Signed-off-by: yuvaldekel <yuvddd05@gmail.com>
@SuperQ
SuperQ requested a review from simonpasquier July 31, 2026 12:28
@simonpasquier

Copy link
Copy Markdown
Contributor

@yuvaldekel #361 (comment) still stands

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support path prefix

3 participants