Skip to content

fix(codespaces): fix reverse proxy warning and enable CSP#2185

Merged
gounthar merged 1 commit into
jenkins-docs:weeklyfrom
gounthar:fix/codespaces-url-and-csp-weekly
Apr 21, 2026
Merged

fix(codespaces): fix reverse proxy warning and enable CSP#2185
gounthar merged 1 commit into
jenkins-docs:weeklyfrom
gounthar:fix/codespaces-url-and-csp-weekly

Conversation

@gounthar

Copy link
Copy Markdown
Collaborator

Summary

Backport of #2184 to the weekly branch. Identical three-file change.

Changes

docker-compose.yaml

Pass CODESPACE_NAME and GITHUB_CODESPACES_PORT_FORWARDING_DOMAIN into the discovery_and_jcasc_modifier container. Docker Compose forwards them transparently from the host; empty string default means no impact on local or CI runs.

dockerfiles/agent-discovery/find-name.sh

When the Codespaces vars are set, update unclassified.location.url to the correct forwarding URL and suppress hudson.diagnosis.ReverseProxySetupMonitor. This runs before the final JCasc reload so the correct URL is active from first startup.

dockerfiles/jenkins.yaml

Add security.contentSecurityPolicy with a policy that covers Jenkins' own UI requirements (inline styles/scripts) while establishing a defined CSP baseline.

Backport of the main branch fix. Same three changes:
- docker-compose.yaml: pass CODESPACE_NAME and
  GITHUB_CODESPACES_PORT_FORWARDING_DOMAIN into the discovery container
- find-name.sh: update unclassified.location.url and suppress
  ReverseProxySetupMonitor when running in Codespaces
- jenkins.yaml: add security.contentSecurityPolicy to clear the CSP
  administrative monitor warning

Signed-off-by: Bruno Verachten <gounthar@gmail.com>
@github-actions github-actions Bot added the repo label Apr 21, 2026
@gounthar gounthar merged commit ac1c42d into jenkins-docs:weekly Apr 21, 2026
2 of 3 checks passed
@gounthar gounthar deleted the fix/codespaces-url-and-csp-weekly branch April 21, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant