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

nginx ssl config example uses deprecated directive (nginx: [emerg] unknown directive "ssl") #5120

Open
majamee opened this issue Apr 25, 2024 · 4 comments

Comments

@majamee
Copy link

majamee commented Apr 25, 2024

The config example for nginx ssl is outdated:
https://github.com/Countly/countly-server/blob/master/bin/config/nginx.server.ssl.conf

If you are trying to use the above example file to start an nginx server, this will fail by now with any more recent nginx version with the error message:
nginx: [emerg] unknown directive "ssl"

In order to fix it:

  • Change line 14 from listen 443; to listen 443 ssl;
  • Remove line 20 completely ssl on;

That would be it, after these changes the nginx ssl config example becomes usable again :)

@ar2rsawseen
Copy link
Member

Do you know from which Nginx version this happens?

@majamee
Copy link
Author

majamee commented Apr 29, 2024

Good day, for me it was the update to nginx version: nginx/1.26.0

@4nonch
Copy link

4nonch commented Jan 3, 2025

Can confirm such behavior. Al tho i'm not using countly-server but has upgraded nginx to 1.26 version. Tried to search for solution and @majamee issue helped me.

ssl on; line removal seems like a solution. Had it with listen 443 ssl;
Seems like 1.26 version has some important changes in config layout.

@ar2rsawseen
Copy link
Member

@4nonch, thanks for the described changes. Do you want to create a PR for it to seal your contribution? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants