stacklok / toolhive / 26446325560

66%

Build Type

push

github

Committed by web-flow

Commit Message

Address post-approval review feedback on `VirtualMCPServer` authz (#5368)

Follow-ups from the post-approval review of #5290, bundled because they
touch overlapping files:

* Distinguish `AuthzConfigMapInvalid` from `AuthzConfigMapNotFound` on
  `MCPRemoteProxy`, restoring status condition parity with
  `VirtualMCPServer`.
* Gate the `AuthzPrimaryUpstreamProviderDeprecated` warning event on a
  generation transition so it fires once per spec change.
* Align the authz config `version` field on `"1.0"` via a shared
  constant; the inline path previously diverged with `"v1"`.
* Narrow the misleading `ExtractCedarAuthzOptions` doc comment.

Coverage Stats

20 of 20 new or added lines in 3 files covered. (100.0%)

5 existing lines in 2 files now uncovered.

65285 of 99131 relevant lines covered (65.86%)

62.03 hits per line