Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent nginx HTTP Server Detection #1698

Merged
merged 1 commit into from
Apr 13, 2022

Conversation

t3easy
Copy link
Contributor

@t3easy t3easy commented Feb 17, 2022

No description provided.

@J0WI
Copy link
Contributor

J0WI commented Feb 18, 2022

@t3easy
Copy link
Contributor Author

t3easy commented Feb 18, 2022

The config is taken from here: https://github.com/nextcloud/documentation/blob/master/admin_manual/installation/nginx-root.conf.sample

@J0WI Should I send a PR to the documentation first?
The setting just hides the nginx version, like

https://github.com/nextcloud/documentation/blob/master/admin_manual/installation/nginx-root.conf.sample#L60

does for the php version.

@J0WI
Copy link
Contributor

J0WI commented Feb 20, 2022

Should I send a PR to the documentation first?

Yes please.

@J0WI J0WI added the upstream label Feb 20, 2022
@skjnldsv
Copy link
Member

@t3easy any updates? :)

@t3easy
Copy link
Contributor Author

t3easy commented Apr 13, 2022

@t3easy any updates? :)

See: nextcloud/documentation#8108

@skjnldsv skjnldsv merged commit 58273cd into nextcloud:master Apr 13, 2022
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Apr 26, 2022
Changes:

- https///github.com/nextcloud/docker/commit/00c5180: Merge pull request https///github.com/nextcloud/docker/pull/1737 from J0WI/readme-typos
- https///github.com/nextcloud/docker/commit/a497f03: Runs update.sh
- https///github.com/nextcloud/docker/commit/f511ef2: Minor typos
- https///github.com/nextcloud/docker/commit/b842cb3: Merge pull request https///github.com/nextcloud/docker/pull/1728 from nextcloud/feat/simultaneaous-html-init
- https///github.com/nextcloud/docker/commit/35b8ebf: Allow simultaneous initialization of common html volume
- https///github.com/nextcloud/docker/commit/e76b7ca: Merge pull request https///github.com/nextcloud/docker/pull/1732 from nextcloud/revert-1684-feature/healthcheck
- https///github.com/nextcloud/docker/commit/ca1e773: Revert "Add healthcheck"
- https///github.com/nextcloud/docker/commit/f9d8052: Merge pull request https///github.com/nextcloud/docker/pull/1684 from TheLastProject/feature/healthcheck
- https///github.com/nextcloud/docker/commit/4bb8d00: Create command-rebase.yml
- https///github.com/nextcloud/docker/commit/1bd3a50: Merge pull request https///github.com/nextcloud/docker/pull/1730 from nextcloud/add/summary-mandatory-step
- https///github.com/nextcloud/docker/commit/8355426: Add healthcheck
- https///github.com/nextcloud/docker/commit/58273cd: Merge pull request https///github.com/nextcloud/docker/pull/1698 from t3easy/hide-nginx-infos
- https///github.com/nextcloud/docker/commit/2753dad: Add mandatory summary step for branches protection
- https///github.com/nextcloud/docker/commit/da935d2: Runs update.sh
- https///github.com/nextcloud/docker/commit/e36ca5f: Fix regex for pecl prereleases (https///github.com/nextcloud/docker/pull/1725)
- https///github.com/nextcloud/docker/commit/71f4a94: update.sh: Remove unused paths (https///github.com/nextcloud/docker/pull/1723)
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Apr 26, 2022
Changes:

- https///github.com/nextcloud/docker/commit/3f42156: 23.0.4 stable (https///github.com/nextcloud/docker/pull/1738)
- https///github.com/nextcloud/docker/commit/00c5180: Merge pull request https///github.com/nextcloud/docker/pull/1737 from J0WI/readme-typos
- https///github.com/nextcloud/docker/commit/a497f03: Runs update.sh
- https///github.com/nextcloud/docker/commit/f511ef2: Minor typos
- https///github.com/nextcloud/docker/commit/b842cb3: Merge pull request https///github.com/nextcloud/docker/pull/1728 from nextcloud/feat/simultaneaous-html-init
- https///github.com/nextcloud/docker/commit/35b8ebf: Allow simultaneous initialization of common html volume
- https///github.com/nextcloud/docker/commit/e76b7ca: Merge pull request https///github.com/nextcloud/docker/pull/1732 from nextcloud/revert-1684-feature/healthcheck
- https///github.com/nextcloud/docker/commit/ca1e773: Revert "Add healthcheck"
- https///github.com/nextcloud/docker/commit/f9d8052: Merge pull request https///github.com/nextcloud/docker/pull/1684 from TheLastProject/feature/healthcheck
- https///github.com/nextcloud/docker/commit/4bb8d00: Create command-rebase.yml
- https///github.com/nextcloud/docker/commit/1bd3a50: Merge pull request https///github.com/nextcloud/docker/pull/1730 from nextcloud/add/summary-mandatory-step
- https///github.com/nextcloud/docker/commit/8355426: Add healthcheck
- https///github.com/nextcloud/docker/commit/58273cd: Merge pull request https///github.com/nextcloud/docker/pull/1698 from t3easy/hide-nginx-infos
- https///github.com/nextcloud/docker/commit/2753dad: Add mandatory summary step for branches protection
- https///github.com/nextcloud/docker/commit/da935d2: Runs update.sh
- https///github.com/nextcloud/docker/commit/e36ca5f: Fix regex for pecl prereleases (https///github.com/nextcloud/docker/pull/1725)
- https///github.com/nextcloud/docker/commit/71f4a94: update.sh: Remove unused paths (https///github.com/nextcloud/docker/pull/1723)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants