Skip to content

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

Merged
kesselb merged 1 commit into
stable28from
backport/45765/stable28
Jul 3, 2024
Merged

[stable28] fix(settings): Use axios directly for health check to preven URL sanitizing#45770
kesselb merged 1 commit into
stable28from
backport/45765/stable28

Conversation

@backportbot

@backportbotbackportbotBot commented Jun 10, 2024

Copy link
Copy Markdown

Backport of #45765

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot
backportbotBot requested review from Pytal, grnd-alt and susnuxJune 10, 2024 19:51
@backportbotbackportbotBot added the 3. to review Waiting for reviews label Jun 10, 2024
@backportbotbackportbotBot added this to the Nextcloud 28.0.7 milestone Jun 10, 2024
@susnux

Copy link
Copy Markdown
Contributor

/compile amend/

@susnux
susnuxforce-pushed the backport/45765/stable28 branch from bf05242 to bed7a7cCompareJune 11, 2024 19:41
@susnux
susnux marked this pull request as ready for review June 11, 2024 19:41
@susnuxsusnux added the bug label Jun 11, 2024
This was referenced Jun 12, 2024
@blizzzblizzz mentioned this pull request Jun 25, 2024
…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>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@kesselb
kesselbforce-pushed the backport/45765/stable28 branch from e49be75 to 6e52a19CompareJuly 3, 2024 13:10
@kesselb
kesselb merged commit f9ec46d into stable28Jul 3, 2024
@kesselb
kesselb deleted the backport/45765/stable28 branch July 3, 2024 16:30
@AltahrimAltahrim mentioned this pull request Jul 10, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@susnux@kesselb@blizzz