Skip to content
This repository has been archived by the owner on May 20, 2022. It is now read-only.

nginx: [warn] the "ssl" directive is deprecated #451

Closed
caugner opened this issue Feb 20, 2020 · 0 comments · Fixed by #452
Closed

nginx: [warn] the "ssl" directive is deprecated #451

caugner opened this issue Feb 20, 2020 · 0 comments · Fixed by #452
Assignees
@caugner

Comments

@caugner
Copy link
Contributor

caugner commented Feb 20, 2020

Problem: I'm getting the following warning when starting mattermost-docker:

nginx: [warn] the "ssl" directive is deprecated, use the "listen ... ssl" directive instead in /etc/nginx/conf.d/mattermost.conf:15

Solution: The ssl on; line should be removed here:

mattermost-docker/web/mattermost-ssl

Lines 13 to 15 in dd1ecde

listen 443 ssl http2;
ssl on;

@caugner caugner mentioned this issue Feb 20, 2020
Merged
DSchalla pushed a commit that referenced this issue Mar 3, 2020
@caugner
Remove deprecated nginx ssl directive (#451) (#452)
1427315
lvnilesh added a commit to beacloudgenius/mattermost-docker that referenced this issue Mar 5, 2020
@lvnilesh
Merge commit '1427315ef927914e0ee03fe230cc4a1b045ebdb7'
a9eb3c8 
* commit '1427315ef927914e0ee03fe230cc4a1b045ebdb7':
  Remove deprecated nginx ssl directive (mattermost#451) (mattermost#452)
  bump mm prod app to 5.20.1
  bump mm prod app to 5.20.0
  bump to 5.19.1
  Update Dockerfile (mattermost#448)
  bump to 5.19.0 (mattermost#444)
  Installing missing package pv (mattermost#443)
  bump to 5.18.1 (mattermost#441)
  Fixes for the smooth deployment of Team edition. (mattermost#438)
  bump to 5.18.0
  Bug fixes (mattermost#427)
  bump to 5.17.1
  bump to 5.17.0
  Fix for letsencrypt, where to run on 443 you need to setcap cap_net_bind_service=+ep ./bin/mattermost. Note, this is needed internally from the container even if you set the correct CAP_NET_BIND_SERVICE in the docker compose env (mattermost#425)
  bump to 5.16.3
  save the sql datasource into an env variable in the app container (mattermost#421)
  bump to 5.16.2
  bump to 5.16.1
  bump to 5.16.0
  bump to 5.15.1

# Conflicts:
#	app/Dockerfile
#	docker-compose.yml
nickygerritsen added a commit to Lets-Talk-NL/mattermost-docker that referenced this issue Jun 29, 2020
@nickygerritsen
Merge remote-tracking branch 'upstream/master'
c6ec708 
* upstream/master: (54 commits)
  bump to 5.24.2
  bump MM to 5.24.1
  bump MM to 5.24.0
  Update Dockerfile (mattermost#476)
  Add ?src=docker-app query parameter to identify download source (mattermost#475)
  bump docker app to 5.23.0
  Fix : certificate signed by unknown authority refs : mattermost/mattermost#13059 (mattermost#474)
  bump app dockerfile to 5.22.3
  Fixing docker-compose build process (mattermost#471)
  bump mm to 5.22.1
  Update Dockerfile (mattermost#467)
  docker-compose: fix DB Docker building error  mattermost#463 related wal-e 1.0.0 and azure (mattermost#464)
  allow uncommenting for team edition (mattermost#433)
  Fix the linking of the nginx configuration file (mattermost#458)
  docker-compose: remove unused ENV variable (mattermost#456)
  bump mm prod app to 5.21.0
  bump mm prod app to 5.20.2
  web entrypoint: forcefully link existing config (mattermost#454)
  Remove deprecated nginx ssl directive (mattermost#451) (mattermost#452)
  bump mm prod app to 5.20.1
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@caugner caugner

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removes the deprecated nginx ssl directive (#451)
1 participant
@caugner