search

standardnotes / docs

[Moved to https://github.com/standardnotes/app/tree/main/packages/docs] Documentation for Standard Notes users and developers | https://docs.standardnotes.com
https://docs.standardnotes.com
GNU Affero General Public License v3.0
7 stars 23 forks source link

Nginx documentation error #44

Open Git5000 opened 3 years ago

Git5000 commented 3 years ago

On site Securing HTTP traffic of your Sync server https://docs.standardnotes.org/self-hosting/https-support/ the command for testing nginx must be sudo nginx -t instead of nginx -t otherwise nginx will report strange and confusing errors for a correct setup.

karolsojko commented 3 years ago

As mentioned in here https://docs.standardnotes.org/self-hosting/docker/

We recommend setting up standalone infra as root user due to Docker volume issues - thus the sudo command is not needed in the instructions

Git5000 commented 3 years ago

I would add it anyway for two reasons:

  1. The line before you do use sudo (sudo service nginx restart) - this would then also not be necessary but it is there
  2. The docker can be installed as non root user just fine. But the nginx -tcommand is then very confusing at it runs but produces tons of errors which are only related to the non root status but you think all went wrong