Skip to content

Added relevant requirement to readme-vars.yml#27

Merged
thelamer merged 1 commit intolinuxserver:masterfrom
oilervoss:patch-1
Mar 1, 2026
Merged

Added relevant requirement to readme-vars.yml#27
thelamer merged 1 commit intolinuxserver:masterfrom
oilervoss:patch-1

Conversation

@oilervoss
Copy link
Contributor


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

Description:

I added to the container requirements paragraph the @thelamer words: "The entire stack is not designed to run behind a reverse proxy it is meant to be directly exposed to the internet." to readme-vars.yml

Benefits of this PR and context:

That information is important because this container is a exceptional a exceptional case compared to other LinuxServer ones.
I have every single other LinuxServer containeir running behind Caddy without any setbacks. I had been struggling with the Sealskin connection until I found the issue #6 . I wish I had read clearly that before.

How Has This Been Tested?

Once I removed SealSkin from Caddy and exposed it directly, everything is running smoothly.

Source / References:

@thelamer mentioned it on #6

Updated app setup block with additional information regarding container requirements. 

I had been struggling with the Sealskin connection until I found the issue linuxserver#6 . I wish I had read clearly that before as I have everything else (40) from LinuxServer running behind Caddy.

I added to the container requirements paragraph the @thelamer words:
"The entire stack is not designed to run behind a reverse proxy it is meant to be directly exposed to the internet."
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for opening this pull request! Be sure to follow the pull request template!

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/sealskin/0.1.37-pkg-66cb85ad-dev-cc2fa97afe8f9218f9a7dfb0c19173fdc1c3e5a7-pr-27/index.html
https://ci-tests.linuxserver.io/lspipepr/sealskin/0.1.37-pkg-66cb85ad-dev-cc2fa97afe8f9218f9a7dfb0c19173fdc1c3e5a7-pr-27/shellcheck-result.xml

Tag Passed
amd64-0.1.37-pkg-66cb85ad-dev-cc2fa97afe8f9218f9a7dfb0c19173fdc1c3e5a7-pr-27
arm64v8-0.1.37-pkg-66cb85ad-dev-cc2fa97afe8f9218f9a7dfb0c19173fdc1c3e5a7-pr-27

@github-project-automation github-project-automation bot moved this from PRs to PRs Approved in Issue & PR Tracker Mar 1, 2026
@thelamer thelamer merged commit 8679022 into linuxserver:master Mar 1, 2026
3 checks passed
@LinuxServer-CI LinuxServer-CI moved this from PRs Approved to Done in Issue & PR Tracker Mar 1, 2026
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.

3 participants