Skip to content

Commit

Permalink
Fix wrangler docker again (#231)
Browse files Browse the repository at this point in the history
At some point this pattern got changed and I noticed "invalid arguments"
error from Wrangler on starting the container. Swithcing this to
entrypoint and only passing in arguments from the docker compose
configuration to avoid this all.
  • Loading branch information
agjohnson authored Nov 4, 2024
1 parent f1c5618 commit 594de44
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 0 additions & 2 deletions dockerfiles/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,6 @@ services:
readthedocs:
working_dir: /usr/src/app/packages/addons-inject/
command: [
"node_modules/.bin/wrangler",
"dev",
"--log-level=info",
"--host=nginx:8080", # El Proxito on NGINX configuration
"--ip=0.0.0.0",
Expand Down
2 changes: 1 addition & 1 deletion packages/addons-inject/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@ COPY index.js /usr/src/app/packages/addons-inject/
COPY wrangler.toml /usr/src/app/packages/addons-inject/
WORKDIR /usr/src/app/packages/addons-inject/
RUN npm install --omit dev
CMD ["node_modules/.bin/wrangler", "dev"]
ENTRYPOINT ["node_modules/.bin/wrangler", "dev"]

0 comments on commit 594de44

Please sign in to comment.