Skip to content

Support declared SCIM authentication methods - #170

Open
pranems wants to merge 1 commit into
AzureAD:masterfrom
pranems:feature/auth-methods-20261016
Open

pranems wants to merge 1 commit into
AzureAD:masterfrom
pranems:feature/auth-methods-20261016

Conversation

@pranems

@pranems pranems commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Higher-priority authentication-only change, split from the larger authentication/reporting work.

  • Declare available authentication methods explicitly and require every selected method to pass (AND).
  • Treat undeclared methods as Not applicable without issuing their token requests.
  • Select direct SCIM transport independently within the declaration.
  • Acquire and refresh OAuth/federation access tokens at runtime, without static-token fallback.
  • Keep bearer-only validation from establishing gallery authentication readiness.
  • Update the canonical onboarding agent and focused authentication documentation.
  • Advance the authentication bundle from 13.0 to 14.0.

Scope

The portable reporting exporter, reporting regression suite, automatic reporting-startup events, and reporting integration are intentionally excluded. The reporting follow-up is dependent and will be reviewed separately.

This commit preserves the intended functional HTTP contracts apart from the authentication changes. It does not claim to fix unrelated endpoint or group-functional failures.

Verification

  • 6,075 offline authentication assertions passed, including all 99 direct SCIM request guards.
  • JSON/PowerShell syntax, documentation links, exact LF version bytes, and source/parameter version equality checked.
  • Split/isolation checks confirmed that no reporting implementation leaked into this branch.

Fresh validation status

Draft pending the new, agent-based, first-time deployment and validation run using fresh Logic App and Entra resources. The authentication session exclusively owns this run; no live federation certification is claimed for an OAuth-only environment.

Historical results from the earlier combined bundle are not certification of this separated commit. Any genuine functional failures will remain visible rather than being converted into authentication success or readiness.

Review order

Review this authentication change first. The reporting review is the reporting-only stacked draft pranems#1, because contributors cannot create an upstream authentication base branch with read-only upstream access.

Separate selected-method readiness and refreshable transport from portable reporting. Preserve existing functional HTTP contracts and fail closed for selected authentication failures.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant