Skip to content

feat: Add Helm chart support for webApp configuration#313

Open
clement0010 wants to merge 1 commit into
feat/ct/support-http-web-appfrom
feat/ct/helm-webapp-config
Open

feat: Add Helm chart support for webApp configuration#313
clement0010 wants to merge 1 commit into
feat/ct/support-http-web-appfrom
feat/ct/helm-webapp-config

Conversation

@clement0010
Copy link
Copy Markdown
Contributor

@clement0010 clement0010 commented May 29, 2026

Branched out from #310

Related Tickets

Issue: #309

Changes

  • Add webApp section to values.yaml with enabled and headers fields
  • Add webApp property to values.schema.json with validation (boolean enabled, object headers with string values)
  • Add Helm test suite with 3 test cases covering all rendering paths

🤖 Generated with Claude Code

@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.66%. Comparing base (d37160b) to head (d29d210).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@                      Coverage Diff                      @@
##           feat/ct/support-http-web-app     #313   +/-   ##
=============================================================
  Coverage                         85.66%   85.66%           
=============================================================
  Files                                40       40           
  Lines                              2825     2825           
=============================================================
  Hits                               2420     2420           
  Misses                              279      279           
  Partials                            126      126           
Flag Coverage Δ
integration 52.83% <ø> (ø)
unit 78.47% <ø> (+0.35%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@clement0010 clement0010 requested review from Copilot and minhtule and removed request for Copilot May 29, 2026 07:26
Expose the webApp protocol handler in the Helm chart so users can enable
HTTP web app proxying and configure custom header injection via values.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@clement0010 clement0010 force-pushed the feat/ct/helm-webapp-config branch from b6e8949 to d29d210 Compare May 29, 2026 07:49
Copy link
Copy Markdown
Contributor

@minhtule minhtule left a comment

Choose a reason for hiding this comment

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

LGTM 👍

minhtule

This comment was marked as duplicate.

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