Skip to content

fix(web): upgrade libxslt to 1.1.43-r4 (unblocks v0.4.0 release) - #866

Merged
ValgulNecron merged 1 commit into
masterfrom
fix/web-image-libxslt
Nov 11, 2026
Merged

ValgulNecron merged 1 commit into
masterfrom
fix/web-image-libxslt

Conversation

@ValgulNecron

@ValgulNecron ValgulNecron commented Nov 10, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

The v0.4.0 release run (42924933170) failed only in build & push images (web): Trivy found HIGH CVE-2026-87636 and CVE-2026-9947 in libxslt 1.1.43-r3 (alpine 3.24.2, amd64 + arm64), fixed in 1.1.43-r4. Because of that, the chart and GitHub release jobs were skipped.

This adds 'libxslt>=1.1.43-r4' to the existing apk upgrade line in web/Dockerfile, the same pattern as #816 and #845. 1.1.43-r4 is in the alpine v3.24 main index.

Test plan

  • CI image build + Trivy scan of the web image on this PR.

Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the application’s runtime image to include a newer XSLT library, with version 1.1.43-r4 or later. This brings the image’s bundled library up to date with fixes available in Alpine 3.24.

The v0.4.0 release run failed the Trivy gate on the web image: the
nginx alpine base ships libxslt 1.1.43-r3 with HIGH CVE-2026-87636 and
CVE-2026-9947, both fixed in 1.1.43-r4 (alpine 3.24 main).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Claude <noreply@anthropic.com>
@ValgulNecron ValgulNecron added area: web React dashboard type: security Security hardening or vulnerability fix labels Nov 10, 2026 — with Claude
@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The web Dockerfile adds a minimum version constraint for libxslt. The vulnerability comment now lists additional CVEs and describes Alpine 3.24 fixes.

Changes

Web image package update

Layer / File(s) Summary
Package installation and vulnerability comment
web/Dockerfile
The package installation adds libxslt>=1.1.43-r4. The comment lists additional libexpat, musl, and libxslt CVEs and describes Alpine 3.24 fixes. Existing package constraints and the root-to-UID-101 transition remain unchanged.

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Bug fix

Possibly related PRs

Suggested labels: type: fix, type: security, area: web

Merge Risk: Minimal · up to ce21a

No merge-blocking issue is established; the image build and scan remain to be run.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly describes the libxslt upgrade and its release impact.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 07bee47 and ce21a43.

Files selected for processing: 1

  • web/Dockerfile

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Recent review details

Context from checks skipped due to timeout: 66

  • GitHub Check: go (telemetry-receiver / arm64)
  • GitHub Check: go (telemetryschema / arm64)
  • GitHub Check: go (gameproto / amd64)
  • GitHub Check: go (gp-module / amd64)
  • GitHub Check: go (capture-sidecar / amd64)
  • GitHub Check: go (mcp-server / arm64)
  • GitHub Check: go (tunnel / arm64)
  • GitHub Check: go (tunnel / amd64)
  • GitHub Check: go (mcp-server / amd64)
  • GitHub Check: go (svcutil / arm64)
  • GitHub Check: go (gp-module / arm64)
  • GitHub Check: go (sentinel / amd64)
  • GitHub Check: go (sentinel / arm64)
  • GitHub Check: go (telemetry-receiver / amd64)
  • GitHub Check: go (svcutil / amd64)
  • GitHub Check: go (capture-sidecar / arm64)
  • GitHub Check: go (gameaction / arm64)
  • GitHub Check: go (gameaction / amd64)
  • GitHub Check: go (agent / amd64)
  • GitHub Check: go (telemetryschema / amd64)
  • GitHub Check: go (netguard / arm64)
  • GitHub Check: go (api / amd64)
  • GitHub Check: go (audit-syslog-bridge / arm64)
  • GitHub Check: go (i18n / amd64)
  • GitHub Check: go (agent / arm64)
  • GitHub Check: go (i18n / arm64)
  • GitHub Check: go (gameproto / arm64)
  • GitHub Check: go (netguard / amd64)
  • GitHub Check: go (api / arm64)
  • GitHub Check: go (audit-syslog-bridge / amd64)
  • GitHub Check: go (operator / amd64)
  • GitHub Check: go (operator / arm64)
  • GitHub Check: lint (gameaction)
  • GitHub Check: lint (agent)
  • GitHub Check: lint (test/e2e)
  • GitHub Check: lint (svcutil)
  • GitHub Check: lint (mcp-server)
  • GitHub Check: lint (capture-sidecar)
  • GitHub Check: lint (sentinel)
  • GitHub Check: lint (tunnel)
  • GitHub Check: lint (telemetry-receiver)
  • GitHub Check: lint (gp-module)
  • GitHub Check: lint (audit-syslog-bridge)
  • GitHub Check: lint (telemetryschema)
  • GitHub Check: lint (api)
  • GitHub Check: lint (i18n)
  • GitHub Check: lint (gameproto)
  • GitHub Check: lint (netguard)
  • GitHub Check: lint (operator)
  • GitHub Check: api (postgres)
  • GitHub Check: e2e bucket coverage
  • GitHub Check: web
  • GitHub Check: dependency review
  • GitHub Check: go (test/e2e)
  • GitHub Check: workflow lint
  • GitHub Check: web e2e (mock)
  • GitHub Check: capture-sidecar CAP_NET_RAW survival
  • GitHub Check: standalone Compose smoke
  • GitHub Check: build e2e images
  • GitHub Check: vuln (go)
  • GitHub Check: build e2e images (arm64)
  • GitHub Check: docs and repository checks
  • GitHub Check: CodeRabbit
  • GitHub Check: en-XA screenshots (spec 019)
  • GitHub Check: portable-paths
  • GitHub Check: design vs browser visual diff
Additional context used

Code guidelines: 1

CLAUDE.md — configured
Path-based instructions: 1

Applicability reference: AGENTS.md: # Gameplane — AI assistant guide **Scout once, brief many:** one scout reads code/docs/failures and writes a factual brief (exact `file:line`, before/after code, justification).

CodeRabbit inference engine (CLAUDE.md)

Files:

  • web/Dockerfile

@github-actions

Copy link
Copy Markdown

CI Report - ce21a43

run 42927425176 - attempt 1

0 failed - 22 passed - 11 skipped - 28m10s elapsed

Test selection: partial - per-suite table

Coverage

Module Measured Gate Margin
web L95.82 S94.18 B88.49 F91.73 L92 S92 B82 F76 +2.18

Changed

Area Files +/-
web/ 1 +3/-2

Slowest jobs

Job Duration
e2e web live / arm64 (kind) 10m22s
e2e web live / amd64 (kind) 10m21s
build e2e images (arm64) 9m36s
e2e postgres / amd64 (kind) 8m34s
build e2e images 7m17s

e2e buckets run

api-agent, api-auth, api-mods, api-rbac, api-roles, backup-preparation, management-placement, monitoring, multicluster, network-isolation, operator, optional-components, postgres-availability, restricted-workloads, telemetry, upgrade

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: web React dashboard type: security Security hardening or vulnerability fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants