Skip to content

(PTFE-3070) Fix redirection web ui#215

Open
matthiasL-scality wants to merge 1 commit intomainfrom
PTFE-3070-resolve-redirection-web-ui
Open

(PTFE-3070) Fix redirection web ui#215
matthiasL-scality wants to merge 1 commit intomainfrom
PTFE-3070-resolve-redirection-web-ui

Conversation

@matthiasL-scality
Copy link
Contributor

@matthiasL-scality matthiasL-scality commented Mar 19, 2026

Improvements to URL routing:

  • Added an if statement to check if $builds_backend is /redirect and, if so, rewrites build URLs to /redirect/$1. This ensures proper handling of builds that require redirection.

Why ?

  • Clicking a file in the web UI redirected to the root build listing instead of serving the file. This affected all browser clients (Mozilla User-Agent) navigating via /builds/.
  • The rewrite rule introduced in PTFE-3085 used an nginx variable in the replacement string ($builds_backend$1). When $builds_backend resolved to /download/, nginx failed to match the file-download location and fell through to the root listing location instead

@matthiasL-scality matthiasL-scality force-pushed the PTFE-3070-resolve-redirection-web-ui branch from 9f4481e to 9633c13 Compare March 19, 2026 09:20
@matthiasL-scality matthiasL-scality marked this pull request as ready for review March 19, 2026 09:23
@matthiasL-scality matthiasL-scality requested a review from a team as a code owner March 19, 2026 09:23
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.

1 participant