AB#4182: Secure FromHttpRequest trigger node: validate access token and required role - #6
Merged
mmjakubpoddebniak merged 1 commit intoAug 14, 2026
Conversation
The operator projects an identity authority into every workload it deploys, but this values file never set one, so adapters fell back to their compiled-in loopback default, refused it, and disabled JWT bearer authentication entirely - every caller of a secured FromHttpRequest@2 route was rejected with 401 no matter how valid the token. The chart has carried `operator.authUri` since the trigger was secured; only the value was missing here. It takes the same public identity URI the reporting values already use, so the CoreDNS rewrite resolves it in-cluster and the root CA makes the certificate trusted. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Sets the missing identity authority (operator.authUri) in the Communication Operator Helm values so operator-deployed adapters can validate bearer tokens for secured FromHttpRequest@2 routes in local kind installs.
Changes:
- Add
operator.authUri: "https://identity.127-0-0-1.nip.io"to the operator values. - Document the failure mode when the authority is not provided (adapters disable JWT auth and secured routes return
401).
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
mmjakubpoddebniak
marked this pull request as ready for review
August 14, 2026 11:40
mmjakubpoddebniak
deleted the
dev/int64/getting-started-operator-identity-authority
branch
August 14, 2026 11:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The Communication Operator projects an identity authority into every workload it deploys, and an adapter uses it to validate the bearer tokens that a secured
FromHttpRequest@2route requires. This values file never set one. The operator therefore passed nothing on, each adapter fell back to its compiled-in loopback default, refused to use it, and disabled JWT bearer authentication altogether - so every caller of a secured route was rejected with401no matter how valid its token was. The chart has carriedoperator.authUrisince the trigger was secured; only the value was missing here, which is why a freshly installed cluster looked healthy while every authenticated pipeline call failed. Found on a from-scratch 3.4.90 install, where the adapter logged exactly that condition and the route rejected a valid operator token until the authority was supplied.What changed
operator.authUriis set tohttps://identity.127-0-0-1.nip.io, the same public identity URIreporting-values.yamlin this repo already uses, so the value stays consistent across the install rather than introducing a second spelling.*.127-0-0-1.nip.ioto ingress-nginx, and the root CA makes the certificate trusted.OPERATOR__AUTHURI. Verified on a live cluster too - once the operator had the value, the adapter stopped disabling authentication and a secured route accepted a valid token.