Skip to content

fix: bind OAuth credentials to trusted API bases - #19

Open
arvid-berndtsson wants to merge 2 commits into
OpenVanta:mainfrom
arvid-berndtsson:fix/api-base-credential-safeguards
Open

arvid-berndtsson wants to merge 2 commits into
OpenVanta:mainfrom
arvid-berndtsson:fix/api-base-credential-safeguards

Conversation

@arvid-berndtsson

Copy link
Copy Markdown
Contributor

Summary

  • Restrict OAuth token requests to approved Vanta API hosts unless a custom HTTPS host is explicitly trusted during login.
  • Bind stored OAuth credentials and cached access tokens to a normalized API base, and reject destination mismatches.
  • Prevent explicit credentials from reusing or persisting cached tokens; block token-request redirects and redact response bodies from errors.

Verification

  • npm test: 24 passed.
  • npm run typecheck passed.
  • git diff --check passed.

This is separate from PR #16, which contains only the approved agent-output terminal-control sanitization.

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