Skip to content

refactor(calm-hub-ui): improve authorization header handling when hosted behind ADC/reverse proxy#1780

Merged
jpgough-ms merged 9 commits intofinos:mainfrom
challamani-ms:feature/#1751
Nov 6, 2025
Merged

refactor(calm-hub-ui): improve authorization header handling when hosted behind ADC/reverse proxy#1780
jpgough-ms merged 9 commits intofinos:mainfrom
challamani-ms:feature/#1751

Conversation

@challamani-ms
Copy link
Contributor

@challamani-ms challamani-ms commented Nov 3, 2025

Description

This change improves how the Calm Hub UI handles the Authorization header, when the application is deployed behind an Application Delivery Controller (ADC) or reverse proxy, calm-hub-ui doesn't require integration with oidc-client.

Added readme instructions to resolve “No name matching certificate found” exceptions, which can occur when the backend service hostname does not match the certificate CN/SAN.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎨 Code style/formatting changes
  • ♻️ Refactoring (no functional changes)
  • ⚡ Performance improvements
  • ✅ Test additions or updates
  • 🔧 Chore (maintenance, dependencies, CI, etc.)

Affected Components

  • CLI (cli/)
  • Shared (shared/)
  • CALM Widgets (calm-widgets/)
  • CALM Hub (calm-hub/)
  • CALM Hub UI (calm-hub-ui/)
  • Documentation (docs/)
  • VS Code Extension (calm-plugins/vscode/)
  • Dependencies
  • CI/CD

Commit Message Format ✅

Testing

  • I have tested my changes locally
  • I have added/updated unit tests
  • All existing tests pass

Checklist

  • My commits follow the conventional commit format
  • I have updated documentation if necessary
  • I have added tests for my changes (if applicable)
  • My changes follow the project's coding standards

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 3, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@challamani-ms challamani-ms marked this pull request as draft November 4, 2025 00:14
@challamani-ms challamani-ms marked this pull request as ready for review November 4, 2025 16:55
aidanm3341
aidanm3341 previously approved these changes Nov 5, 2025
@jpgough-ms jpgough-ms merged commit 2985070 into finos:main Nov 6, 2025
12 checks passed
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.

3 participants