Skip to content

[stable31] fix(settings): Use axios directly for health check to preven URL sanitizing - #52440

Merged
solracsf merged 2 commits into
stable31from
backport/52439/stable31
Apr 25, 2025
Merged

[stable31] fix(settings): Use axios directly for health check to preven URL sanitizing#52440
solracsf merged 2 commits into
stable31from
backport/52439/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented Apr 25, 2025

Copy link
Copy Markdown

…tizing
When using Apache with `DirectorySlash` it will respond with 301 and the URL with trailing slash.
But when using traefik as the reverse proxy it can not rewrite redirects, this leads to the problem
that the Apache response is using HTTP in the redirect but the real server (traefik) is only listening on HTTPS.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@backportbot
backportbotBot requested review from susnux and szaimenApril 25, 2025 15:46
@backportbotbackportbotBot added the 3. to review Waiting for reviews label Apr 25, 2025
@backportbotbackportbotBot added this to the Nextcloud 31.0.5 milestone Apr 25, 2025
@solracsf

Copy link
Copy Markdown
Member

/compile

@solracsf
solracsf enabled auto-merge April 25, 2025 16:02
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

@kesselbkesselb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@solracsf
solracsf merged commit e6044f3 into stable31Apr 25, 2025
@solracsf
solracsf deleted the backport/52439/stable31 branch April 25, 2025 16:19
@blizzzblizzz mentioned this pull request May 5, 2025
10 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@solracsf@kesselb@susnux@nextcloud-command