Skip to content

fix(visualize): infer auth type from YAML and legacy proxy fields#41

Merged
fmenesesg merged 1 commit into
mainfrom
fix/visualize-auth-inference
Jun 11, 2026
Merged

fix(visualize): infer auth type from YAML and legacy proxy fields#41
fmenesesg merged 1 commit into
mainfrom
fix/visualize-auth-inference

Conversation

@fmenesesg

@fmenesesg fmenesesg commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Proposed change

Fix auth matrix showing unknown for exports that omit proxy.auth_type. The visualizer now resolves authentication from toolbox product YAML and legacy Admin API proxy fields.

Type of change

  • Bugfix (non-breaking)

RHCL checklist

  • Tests added or updated (go test ./... or mvn test / npm test)
  • CI does not introduce -DskipTests without justification
  • No secrets in diff (tokens, kubeconfigs, OIDC secrets)
  • PR description and docs in English
  • Tested in local lab if applicable (seed → export → analyze)

Test plan

  • go test ./internal/visualize/...
  • Validated against a large tenant export: 101 products, 0 unknown auth (72 App ID + App Key, 29 API Key)

Additional information

Made with Cursor

Resolve API Key, App ID + App Key, and OIDC from toolbox product YAML and
Admin API proxy sidecars when auth_type is absent from proxy.json.

Co-authored-by: Cursor <cursoragent@cursor.com>
@fmenesesg fmenesesg requested a review from pcastelo as a code owner June 11, 2026 21:36
@fmenesesg fmenesesg added the type:bugfix Non-breaking bug fix label Jun 11, 2026
@fmenesesg fmenesesg merged commit 009300b into main Jun 11, 2026
1 check passed
@fmenesesg fmenesesg deleted the fix/visualize-auth-inference branch June 11, 2026 21:40
fmenesesg added a commit that referenced this pull request Jun 11, 2026
…erence

fix(visualize): infer auth type from YAML and legacy proxy fields
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bugfix Non-breaking bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] Infer auth type from proxy legacy fields and product YAML

1 participant