Skip to content

docker-compose.yml has an error #12

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

Open
quteibah opened this issue Feb 24, 2025 · 3 comments
Open

docker-compose.yml has an error #12

quteibah opened this issue Feb 24, 2025 · 3 comments

Comments

@quteibah
Copy link

Hi. please remove or review this part of the .yml file in line 38:

- 'traefik.http.routers.n8n.rule=PathPrefix(/${SUBFOLDER}{regex:$$|/.*})'

I don't know why but it gives 404 with that regex but when removed, like this:

- 'traefik.http.routers.n8n.rule=PathPrefix(/${SUBFOLDER})'

works like a charm.

@netroy netroy changed the title docker-composer.yml has an error docker-compose.yml has an error Mar 18, 2025
@gwbischof
Copy link

gwbischof commented Apr 18, 2025

I hit is error also with the SSL docker-compose example.

I think the port mapping for n8n should be updated also like this:

  n8n:
    image: docker.n8n.io/n8nio/n8n:1.89.2
    expose:
      - "5678"

The port mapping as it currently is will map the 5678 port on your host to the 5678 port in the container, thereby bypassing the Traefik proxy. I don't even think it is necessary to expose the port either.

@gwbischof
Copy link

gwbischof commented Apr 19, 2025

Mounting the docker sock on the n8n container is a security risk. A quick web search revealed a bunch of articles advocating for not doing this.

- /var/run/docker.sock:/var/run/docker.sock

I'm not sure if there are some features that require it, I haven't found any specifics in the docs.
It seems to work just fine if you remove that line from the docker config.

I'm taking a guess.. Maybe it's only needed if you are trying to manage containers from your workflow.

@netroy
Copy link
Member

netroy commented Apr 20, 2025

That looks like a copy-paste issue. n8n container does not need access to the docker socket. I'll get that fixed.

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