getting-started.md improvements #10

Closed
opened 2025-12-29 15:29:37 +01:00 by adam · 3 comments
Owner

Originally created by @bellwood on GitHub (Jun 27, 2016).

First off, my apologies this isn't in a pull request

  1. "ALLOWED_HOSTS"

I had to add 127.0.0.1 as nginx is calling it for the proxy

  1. gunicorn Configuration

You may want to elude to the command path needing to be checked, my path was sans local:

command = '/usr/bin/gunicorn'
Originally created by @bellwood on GitHub (Jun 27, 2016). First off, my apologies this isn't in a pull request 1) "ALLOWED_HOSTS" I had to add 127.0.0.1 as nginx is calling it for the proxy 2) gunicorn Configuration You may want to elude to the command path needing to be checked, my path was sans local: ``` command = '/usr/bin/gunicorn' ```
adam closed this issue 2025-12-29 15:29:37 +01:00
Author
Owner

@jeremystretch commented on GitHub (Jun 27, 2016):

I had to add 127.0.0.1 as nginx is calling it for the proxy

This shouldn't be necessary. Do you have the following under location / in your nginx config?

proxy_set_header X-Forwarded-Host $server_name;

You may want to elude to the command path needing to be checked, my path was sans local

Looks like the default install location for Ubuntu changed recently. I'll update the doc to remove local from the path. Thanks for the heads up.

@jeremystretch commented on GitHub (Jun 27, 2016): > I had to add 127.0.0.1 as nginx is calling it for the proxy This shouldn't be necessary. Do you have the following under `location /` in your nginx config? ``` proxy_set_header X-Forwarded-Host $server_name; ``` > You may want to elude to the command path needing to be checked, my path was sans local Looks like the default install location for Ubuntu changed recently. I'll update the doc to remove `local` from the path. Thanks for the heads up.
Author
Owner

@bellwood commented on GitHub (Jun 27, 2016):

@jeremystretch it looks like adding localhost isn't required - i had changed that and then removed local at the same time

removing 127.0.0.1 now and restarting supervisor leaves me with a working instance so I jumped the gun on that one

So far so good... I really like the work you've done here =)

@bellwood commented on GitHub (Jun 27, 2016): @jeremystretch it looks like adding localhost isn't required - i had changed that and then removed local at the same time removing 127.0.0.1 now and restarting supervisor leaves me with a working instance so I jumped the gun on that one So far so good... I really like the work you've done here =)
Author
Owner

@bellwood commented on GitHub (Jun 27, 2016):

Looks like this was already addressed in 1cd2086 so I'll close this.

Thanks =)

@bellwood commented on GitHub (Jun 27, 2016): Looks like this was already addressed in 1cd2086 so I'll close this. Thanks =)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#10