Create headscale.subdomain.conf.sample#727
Create headscale.subdomain.conf.sample#727ncraft5 wants to merge 2 commits intolinuxserver:masterfrom
Conversation
|
This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions. |
| # enable for Authentik (requires authentik-location.conf in the location block) | ||
| #include /config/nginx/authentik-server.conf; | ||
|
|
||
| location / { |
There was a problem hiding this comment.
Please add a /api location endpoint without the auth comments, you can take a look at other samples as a reference.
There was a problem hiding this comment.
@quietsy Actually /api location is not needed for this proxy to work. I have it in production already for long time, works perfectly fine.
Please let me know, what do you think.
There was a problem hiding this comment.
It's not needed but it's our standard way of writing confs to avoid api issues when enabling authentik/authelia.
There was a problem hiding this comment.
@quietsy /api location is not added because in example file _template.subdomain.conf.sample mentioned that, if /api is not required for headscale it should not be included.
As this comment in the file confusing for me, can you please explain, as you have said it should be included.
Otherpoint that headscale is really not compatible with authentik and authelia, so I would like to also remove that commented lines, because it will be confusing to keep something which will not be compatible with headscale.
Please let me know what do you think? Thanks for you time.
# REMOVE THIS LINE BEFORE SUBMITTING: Some proxies require one or more additional location blocks for things like API or RPC endpoints.
# REMOVE THIS LINE BEFORE SUBMITTING: If the proxy you are making a sample for does not require an additional location block please remove the commented out section below.
# location ~ (/<container_name>)?/api {
# include /config/nginx/proxy.conf;
# include /config/nginx/resolver.conf;
# set $upstream_app <container_name>;
# set $upstream_port <port_number>;
# set $upstream_proto <http or https>;
# proxy_pass $upstream_proto://$upstream_app:$upstream_port;
#
# # REMOVE THIS LINE BEFORE SUBMITTING: Additional proxy settings such as headers go below this line, leave the blank line above.
# }
|
This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request is locked due to inactivity |
3 participants
Description
Headscale Nginx configs added.
Benefits of this PR and context
Headscale Nginx configs were missing, so it is added. It will help everyone who wants to configure headscale.
How Has This Been Tested?
I have setup Headscale with tailscale client on phone and laptop to make sure that config works.
Source / References
https://headscale.net/stable/ref/integration/reverse-proxy/?h=nginx