Skip to content

DOCU-1441: Document JSON object context support for Conditional Data Access#290

Open
samsternberg wants to merge 1 commit intomainfrom
feature/DOCU-1441-conditional-data-access-docs
Open

DOCU-1441: Document JSON object context support for Conditional Data Access#290
samsternberg wants to merge 1 commit intomainfrom
feature/DOCU-1441-conditional-data-access-docs

Conversation

@samsternberg
Copy link
Copy Markdown
Collaborator

Summary

  • Documents the existing ctx JSON object support in bearer token and signed data token generation
  • The Node SDK already supports ctx as string | Record<string, any> (shipped in SK-2391/PR SK-2422 allow context as object in vault creds #266), but the README and samples only showed string usage
  • Adds README examples showing both string context and JSON object context with CEL expression mapping
  • Updates sample files to demonstrate both approaches

Replaces #289 (which was created from a fork and couldn't run CI).

Test plan

  • Verify README renders correctly on GitHub
  • Verify sample code compiles with tsc
  • No code changes — documentation only

Refs DOCU-1441

… Access

The Node SDK already supports ctx as string | Record<string, any> for
bearer tokens and signed data tokens (shipped in SK-2391/PR #266), but
the README and samples only showed string usage. Add documentation and
examples for JSON object context to support Conditional Data Access
CEL expressions like request.context.role == 'admin'.

Refs DOCU-1441
Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

Gitleaks Findings: No secrets detected. Safe to proceed!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging.

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