Skip to content

Update redirect location and use front controller#72

Merged
drizuid merged 3 commits intolinuxserver:masterfrom
mikespub-org:redirect_front
Feb 23, 2026
Merged

Update redirect location and use front controller#72
drizuid merged 3 commits intolinuxserver:masterfrom
mikespub-org:redirect_front

Conversation

@mikespub
Copy link
Contributor

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

Update redirect location and use front controller - see issue #71

Note: this includes merged PR #70 to avoid rebase conflicts

Benefits of this PR and context:

See issue #71

How Has This Been Tested?

Local tests

Source / References:

See issue #71

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/cops/4.2.0-pkg-f75194b1-dev-31dc8357dfd11def4c22cc79ce87fb9ccc0185b7-pr-72/index.html
https://ci-tests.linuxserver.io/lspipepr/cops/4.2.0-pkg-f75194b1-dev-31dc8357dfd11def4c22cc79ce87fb9ccc0185b7-pr-72/shellcheck-result.xml

Tag Passed
amd64-4.2.0-pkg-f75194b1-dev-31dc8357dfd11def4c22cc79ce87fb9ccc0185b7-pr-72
arm64v8-4.2.0-pkg-f75194b1-dev-31dc8357dfd11def4c22cc79ce87fb9ccc0185b7-pr-72

@mikespub
Copy link
Contributor Author

Ready for review

Note: the nginx config for the front controller was inspired from the docker-nextcloud config and modern PHP framework examples that use PATH_INFO

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/cops/4.2.0-pkg-f75194b1-dev-291463f38faf01a10fcdd6278b91beb7417fa3f5-pr-72/index.html
https://ci-tests.linuxserver.io/lspipepr/cops/4.2.0-pkg-f75194b1-dev-291463f38faf01a10fcdd6278b91beb7417fa3f5-pr-72/shellcheck-result.xml

Tag Passed
amd64-4.2.0-pkg-f75194b1-dev-291463f38faf01a10fcdd6278b91beb7417fa3f5-pr-72
arm64v8-4.2.0-pkg-f75194b1-dev-291463f38faf01a10fcdd6278b91beb7417fa3f5-pr-72

Copy link
Member

@drizuid drizuid left a comment

Choose a reason for hiding this comment

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

blind approval as per agreement with dev

@github-project-automation github-project-automation bot moved this from PRs to PRs Approved in Issue & PR Tracker Feb 23, 2026
@drizuid drizuid merged commit af576de into linuxserver:master Feb 23, 2026
2 checks passed
@LinuxServer-CI LinuxServer-CI moved this from PRs Approved to Done in Issue & PR Tracker Feb 23, 2026
@mikespub mikespub deleted the redirect_front branch February 23, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants