Skip to content

feat(auth): add Repo App user authorization - #957

Merged
sabbour merged 1 commit into
devfrom
feat/941-repo-app-user-auth
Aug 27, 2026
Merged

feat(auth): add Repo App user authorization#957
sabbour merged 1 commit into
devfrom
feat/941-repo-app-user-auth

Conversation

@sabbour

@sabbour sabbour commented Aug 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • add Entra-human-bound Repo App authorization begin, browser callback, safe polling, refresh, and revocation endpoints
  • persist only opaque references; enforce PKCE S256, fixed-time callback-cookie validation, atomic transaction claims, stable grant versions, and fail-closed transaction/credential fencing
  • document Repo App configuration and add focused transaction, secrecy, refresh, revocation, and concurrency coverage

Validation

  • npm run validate:layer -- --area dotnet --dotnet-filter "FullyQualifiedName~RepoAppUserAuthorizationServiceTests|FullyQualifiedName~TwoAppPersistenceStoreTests" (27 passed)
  • npm run version:check
  • npm run changeset:check

Reviews

  • Post-implementation code review complete; findings addressed.
  • Seraph-style security review complete; no high-confidence findings.

Note

  • Local docs build could not run because the configured package proxy returned 404 for mermaid@11.17.1; the documentation change is limited to configuration/API references.

Closes #941

Closes #941

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7762866f-fa88-4087-b5df-17f482d36ec0
@sabbour

sabbour commented Aug 27, 2026

Copy link
Copy Markdown
Owner Author

Post-implementation review complete. The code-review pass identified transaction fencing, refresh concurrency, revocation, and post-claim finalization risks; all were addressed and focused validation was rerun. Seraph-style security review reported no high-confidence security findings on the final change.

npm run validate:layer -- --area dotnet --dotnet-filter "FullyQualifiedName~RepoAppUserAuthorizationServiceTests|FullyQualifiedName~TwoAppPersistenceStoreTests" passes (27 tests).

@sabbour
sabbour merged commit 9431463 into dev Aug 27, 2026
15 checks passed
@sabbour
sabbour deleted the feat/941-repo-app-user-auth branch August 27, 2026 13:50
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.

feat: add Repo App user authorization

2 participants