Skip to content

Update default.conf.sample to deny dotfile access#133

Open
nemchik wants to merge 2 commits intomainfrom
deny-dotfile-access
Open

Update default.conf.sample to deny dotfile access#133
nemchik wants to merge 2 commits intomainfrom
deny-dotfile-access

Conversation

@nemchik
Copy link
Copy Markdown
Member

@nemchik nemchik commented Feb 8, 2026

No description provided.

Signed-off-by: Eric Nemchik <eric@nemchik.com>
@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-mastodon:v4.5.6-pkg-33bb70c3-dev-b0c11a308e9488afeb3b1bfdce0e593733d65d0d-pr-133

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-mastodon:v4.5.7-pkg-126d7764-dev-d5586841dde72ad1c3e4386e6f67468bc1908114-pr-133

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-mastodon:v4.5.8-pkg-5106bfc9-dev-5960c4c06744784ba28f334110d0b83a9178ce3a-pr-133

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

Signed-off-by: Eric Nemchik <eric@nemchik.com>
Copilot AI review requested due to automatic review settings May 5, 2026 21:43
@LinuxServer-CI LinuxServer-CI moved this from PRs to PRs Ready For Team Review in Issue & PR Tracker May 5, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the sample Nginx site config for the Mastodon container, primarily to block access to hidden files under the public web root while preserving .well-known access for expected public endpoints.

Changes:

  • Add a broader dotfile-deny rule to default.conf.sample.
  • Add an explicit /.well-known exception ahead of that deny rule.
  • Update the changelog entry for operators who maintain a copied Nginx config.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
root/defaults/nginx/site-confs/default.conf.sample Adds dotfile blocking, a .well-known exception, and new QUIC listen directives.
readme-vars.yml Adds a changelog entry describing the config update for existing users.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +32 to +33
# Allow access to the ".well-known" directory
location ^~ /.well-known {
Comment on lines +23 to +24
listen 443 quic reuseport default_server;
listen [::]:443 quic reuseport default_server;
Comment thread readme-vars.yml
"mastodon:latest" <- Base Images
# changelog
changelogs:
- {date: "08.02.26:", desc: "Existing users should update: site-confs/default.conf - Deny access to all dotfiles."}
@LinuxServer-CI LinuxServer-CI moved this from PRs Ready For Team Review to PRs in Issue & PR Tracker May 5, 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