Skip to content

Conversation

@spike83
Copy link

@spike83 spike83 commented Jan 22, 2026

Implements issue #3728: allow running behind a reverse proxy under a sub path by configuring servlet mappings and UI base href via CONTEXT_PATH.

Implements issue mapfish#3728: allow running behind a reverse proxy under a sub path by configuring servlet mappings and UI base href via CONTEXT_PATH.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for deploying the application behind a reverse proxy under a sub-path by introducing a CONTEXT_PATH environment variable. The implementation modifies servlet URL mappings, copies UI assets to the configured sub-path, and injects base href tags for proper relative link resolution.

Changes:

  • Added CONTEXT_PATH environment variable configuration with path normalization and validation
  • Implemented startup script logic to rewrite web.xml servlet mappings and set up UI assets under the configured sub-path
  • Added documentation explaining CONTEXT_PATH usage and filesystem write requirements

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
docs/src/main/resources/templates/docker.html Documents the new CONTEXT_PATH variable, its behavior, and filesystem write requirements when enabled
core/docker/usr/local/tomcat/bin/docker-start-print Implements path normalization, validation, and applies context path by rewriting web.xml and copying UI assets

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

@sebr72
Copy link
Contributor

sebr72 commented Jan 22, 2026

@spike83 I have assign to review your PR. But it is not successfully building. Could you please fix you PR, then I will review it.

@sebr72
Copy link
Contributor

sebr72 commented Jan 22, 2026

@sbrunner Could we do sthg about Pull request links that never stops running ?

@spike83 spike83 force-pushed the issue-3728-context-path branch from eabe877 to 901814b Compare January 23, 2026 05:27
@sbrunner
Copy link
Member

sbrunner commented Jan 23, 2026

@sebr72 I just blacklist 'issue' to be a Jira project.

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