Skip to content

Commit 80c4a0e

Browse files
davidjgraphclaude
andcommitted
Fix export-server render 500: stop purging Chrome's wget dependency
The render test (#675) returned HTTP 500 on every diagram. Captured logs show Puppeteer failing with "Tried to find the browser at the configured path (/usr/bin/google-chrome-stable), but no executable was found". google-chrome-stable Depends on wget, so the `apt-get remove -y --purge git wget` cleanup uninstalls Chrome along with wget, leaving PUPPETEER_EXECUTABLE_PATH (set in dc57dcc) pointing at a path that no longer exists. This is why the prior fix was necessary but not sufficient. Only purge git (a build-time clone tool, not a Chrome dependency); keep wget. Add a `test -x /usr/bin/google-chrome-stable` build-time guard so a future regression fails the build immediately rather than at render time. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 6fa7cc2 commit 80c4a0e

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

image-export/Dockerfile

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,12 @@ RUN apt-get update \
2727
&& cd /home/pptruser \
2828
&& git clone https://github.com/jgraph/draw-image-export2.git \
2929
&& chown -R pptruser:pptruser /home/pptruser \
30-
&& apt-get remove -y --purge git wget \
30+
# Only purge git (a build-time clone tool). Do NOT purge wget: it is a
31+
# runtime dependency of google-chrome-stable, so removing it uninstalls
32+
# Chrome and leaves PUPPETEER_EXECUTABLE_PATH dangling -> HTTP 500 on render.
33+
&& apt-get remove -y --purge git \
34+
# Fail the build early if the cleanup ever removes Chrome again.
35+
&& test -x /usr/bin/google-chrome-stable \
3136
&& rm -rf /var/lib/apt/lists/*
3237

3338
USER pptruser

0 commit comments

Comments
 (0)