Skip to content

Add mapbender-puppeteer target to Dockerfile#164

Merged
Phocacius merged 6 commits intomasterfrom
feature/puppeteer-docker-image
Mar 18, 2026
Merged

Add mapbender-puppeteer target to Dockerfile#164
Phocacius merged 6 commits intomasterfrom
feature/puppeteer-docker-image

Conversation

@cioddi
Copy link
Member

@cioddi cioddi commented Feb 25, 2026

Build the image

docker build -t mapbender-puppeteer-test --progress=plain --target mapbender-puppeteer .

Start the Container

docker run -p 80:8080 -e MB_PUPPETEER_NO_SANDBOX=true -it mapbender-puppeteer-test:latest

Test in the browser

Navigate to localhost:80 to test the VT-print feature in the browser.

@astroidex astroidex added this to the 4.2.5 milestone Mar 12, 2026
@cioddi
Copy link
Member Author

cioddi commented Mar 17, 2026

The new image version now works after running composer update.

Build the image

docker build -t mapbender-puppeteer-test --progress=plain --target mapbender-puppeteer .

Start the Container

docker run -p 80:8080 -e MB_PUPPETEER_NO_SANDBOX=true -it mapbender-puppeteer-test:latest

Run composer update

Find the container_id using docker ps

docker exec {container_id} bash -c "cd application/ && bin/composer update"

Test in the browser

Navigate to localhost:80 to test the VT-print feature in the browser.

@cioddi cioddi marked this pull request as ready for review March 17, 2026 15:00
@cioddi cioddi requested a review from Phocacius March 17, 2026 15:00
@cioddi
Copy link
Member Author

cioddi commented Mar 17, 2026

I have never used the matrix strategy in GH-actions, but the config in this PR looks good to me (https://docs.github.com/de/actions/how-tos/write-workflows/choose-what-workflows-do/run-job-variations).

@Phocacius Phocacius merged commit 8335b5f into master Mar 18, 2026
@Phocacius Phocacius deleted the feature/puppeteer-docker-image branch March 18, 2026 09:43
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

Successfully merging this pull request may close these issues.

3 participants