Skip to content

fix: mapfish print and tile cache optimizations#793

Open
wp99cp wants to merge 2 commits into
masterfrom
fix/tile-cache-and-print-fixes
Open

fix: mapfish print and tile cache optimizations#793
wp99cp wants to merge 2 commits into
masterfrom
fix/tile-cache-and-print-fixes

Conversation

@wp99cp

@wp99cp wp99cp commented Jun 8, 2026

Copy link
Copy Markdown
Member

Backports the print URL fix and the tile cache reverse proxy load-balancing optimizations to the master branch.

Copilot AI review requested due to automatic review settings June 8, 2026 19:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Backports two production-facing fixes: updating the Mapfish print endpoint to the correct print-app name, and optimizing the Nginx tile-cache reverse proxy to better handle heavy load by introducing upstream load-balancing and connection reuse.

Changes:

  • Update Mapfish print URL to target the swisstopo print-app (/print/swisstopo/report.pdf).
  • Add Nginx upstream pool for wmts*.geo.admin.ch with keepalive + HTTP/1.1 proxying, and tune keepalive settings.
  • Adjust tile-cache image build to create the cache directory and set it owned by the nginx user.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
tile_caching/nginx.conf Adds upstream load-balancing and keepalive-related proxy optimizations for WMTS tile requests.
tile_caching/Dockerfile Ensures the cache directory exists and is owned by nginx in the built image.
backend/automatic_walk_time_tables/map_downloader/create_map.py Fixes Mapfish print endpoint path to use the swisstopo print-app.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tile_caching/nginx.conf
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.

2 participants