Skip to content

dbeaver/pro#11347 Expose verified SSO access claims - #29

Merged
alexander-skoblikov merged 1 commit into
develfrom
dbeaver/pro#11347-verified-access-claims
Nov 2, 2026
Merged

alexander-skoblikov merged 1 commit into
develfrom
dbeaver/pro#11347-verified-access-claims

Conversation

@labanovichttps

Copy link
Copy Markdown
Contributor

Account's private access-context endpoint resolves permissions for tokens targeting different resources. Previously it parsed an unverified audience before calling the verifier with that same audience.

Add verifyAccessToken(token) to return the verified access identity, including its audience, so consumers can enforce the expected resource at the call site. Keep verifyAccessToken(token, expectedAudience) for existing callers. Make verifiedClaims() public with a sanitized DDSsoClientException; callers of that lower-level method must validate token purpose and audience. Signature, issuer, timestamps, access-token purpose and strict single-audience validation remain enforced by the access-token verifier.

Required by https://github.com/dbeaver/datadam/pull/185; merge and publish this API change before the dependent Account change.

Validation: 664 targeted reactor tests passed with the companion Account changes, including signed-token rejection cases and PostgreSQL Account integration tests. Tests cover audience extraction, malformed/missing/multiple audiences, token purpose, signature, issuer and expiry. git diff --check passed.

Related to dbeaver/pro#11347.

@alexander-skoblikov
alexander-skoblikov merged commit 03438b0 into devel Nov 2, 2026
3 checks passed
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.

2 participants