goryn-clade / pathfinder-containers

A Docker-compose solution for Pathfinder mapping tool using Traefik
Other
62 stars 66 forks source link

Nginx Reverse Proxy configuration #25

Closed leedickey closed 2 years ago

leedickey commented 3 years ago

Reverse Proxy configuration file for NGINX or the LinuxServer.io SWAG docker. That can be found here: https://github.com/linuxserver/docker-swag

leedickey commented 3 years ago

I am currently using this configuration in a SWAG docker without issue.

fleischsalatinspace commented 3 years ago

Can you please include

/config/nginx/resolver.conf
/config/nginx/proxy.conf
include /config/nginx/ssl.conf

Looks like linuxserver.io/SWAG related config.

IMO your config looks like it does not support websockets, which is an issue as pathfinder uses them heavily.

leedickey commented 3 years ago

Can you please include

/config/nginx/resolver.conf
/config/nginx/proxy.conf
include /config/nginx/ssl.conf

Looks like linuxserver.io/SWAG related config.

IMO your config looks like it does not support websockets, which is an issue as pathfinder uses them heavily.

The only change from the default compose file is the removal of traefik. Swag is its own docker-compose being used for other services.

Pf-socket isn't modified at all. Maybe I do not understand the questions but there doesn't seem to be a need to expose any additional ports or services other than 80/443.

fleischsalatinspace commented 3 years ago

With the committed file alone it is not possible to set up a working nginx reverse proxy on a standalone vanilla nginx. In order to add a best-practice nginx configuration file to this project, i suggest you add the contents of

/config/nginx/resolver.conf
/config/nginx/proxy.conf
include /config/nginx/ssl.conf

excluding sensitive stuff like URL's etc.

leedickey commented 3 years ago

With the committed file alone it is not possible to set up a working nginx reverse proxy on a standalone vanilla nginx. In order to add a best-practice nginx configuration file to this project, i suggest you add the contents of

/config/nginx/resolver.conf
/config/nginx/proxy.conf
include /config/nginx/ssl.conf

excluding sensitive stuff like URL's etc.

I will provide this in the next couple of days.

fleischsalatinspace commented 2 years ago

@leedickey Any follow up on this?

leedickey commented 2 years ago

I believe I've got the files uploaded and linked to this request. Please let me know if not. Sorry for the delay! Don't have kids.

fleischsalatinspace commented 2 years ago

I think this would be more suited to be in the docs/wiki of the project instead of in the actual git repository. I linked it in the issue https://github.com/goryn-clade/pathfinder-containers/issues/33 as a TODO. I will close this PR now.