Skip to content

fix(settings): Save one HTTP request on enabling an app - #44939

Merged
susnux merged 2 commits into
masterfrom
fix/reduce-http-noise
May 6, 2024
Merged

fix(settings): Save one HTTP request on enabling an app#44939
susnux merged 2 commits into
masterfrom
fix/reduce-http-noise

Conversation

@susnux

Copy link
Copy Markdown
Contributor

Summary

This prevents one HTTP request for most situations (301 from apps/files to apps/files/) and helps users with faulty reverse-proxy configuration.

Checklist

@susnuxsusnux added bug 3. to review Waiting for reviews labels Apr 19, 2024
@susnuxsusnux added this to the Nextcloud 30 milestone Apr 19, 2024
@susnux
susnux requested review from a team, emoral435, sorbaugh and szaimen and removed request for a teamApril 19, 2024 20:04
@susnuxsusnux changed the title fix(settings): Prevent one HTTP request on enabling an appfix(settings): Save one HTTP request on enabling an appApr 19, 2024
@susnux

Copy link
Copy Markdown
ContributorAuthor

/backport to stable29

@susnux

Copy link
Copy Markdown
ContributorAuthor

/backport to stable28

@szaimenszaimen 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.

Not exactly sure how this helps but fine by me

@susnux

Copy link
Copy Markdown
ContributorAuthor

Not exactly sure how this helps but fine by me

We prevent the 301 HTTP response for apps/files -> apps/files/ and directly use the trailing slash.
This is just one HTTP request (due to redirect), but this prevents invalid redirects by webservers (see linked issue).

susnux added 2 commits May 6, 2024 16:12
…ng slash
This prevents one HTTP request for most situations (301 from `apps/files` to `apps/files/`)
and helps users with faulty reverse-proxy configuration.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the fix/reduce-http-noise branch from 75b0c72 to c81887cCompareMay 6, 2024 14:16
@blizzzblizzz mentioned this pull request Jul 24, 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.

[Bug]: Can't enable or disable apps after upgrade to v28.0.4

4 participants

@susnux@nfebe@szaimen@emoral435