-
Notifications
You must be signed in to change notification settings - Fork 11
feat: auth0-ai sdk Token Vault + CIBA renaming updates #105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
6c1b184
to
5a9279c
Compare
--- | ||
title: Call Other's APIs on User's Behalf | ||
description: "[Token Vault](/intro/token-vault) lets your AI agent call external APIs on the user's behalf. We provide SDKs for Python, JavaScript, and popular AI frameworks like LangChain, LlamaIndex, Genkit, and Vercel AI, making it easy and straightforward to get access tokens for your federated connections." | ||
description: "[Token Vault](/intro/token-vault) lets your AI agent call external APIs on the user's behalf. We provide SDKs for Python, JavaScript, and popular AI frameworks like LangChain, LlamaIndex, Genkit, and Vercel AI, making it easy and straightforward to get access tokens for your third-party connections." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in https://auth0.com/docs/secure/tokens/token-vault, we use the terminology "external provider" instead of "third-party connection". Should we align? (probably more a question for docs partners)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"... making it easy to get access tokens to connect with external applications."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i could definitely use some copy/content/batch suggestion help here. rebased PR again w/ the latest today 😄
5a9279c
to
eadd0c2
Compare
if not google_access_token: | ||
raise ValueError( | ||
"Authorization required to access the Federated Connection API" | ||
"Authorization required to access the Token Vault connection API" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be clearer like this?
"Authorization required to access the Token Vault connection API" | |
"Authorization required to access the Google Calendar API" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's this new file? Are you combining other changes to this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch... seems like a stale branch here. Will be sure to remove w/ the next pass here 👍
Description
Adopts new AI SDK naming conventions for Async Auth and Token Vault features.
See internal documentation about this update:
https://oktawiki.atlassian.net/wiki/spaces/~5a46aca46be331292799e4a9/pages/3805906246/AI+SDKs+-+CIBA+and+Token+Vault+naming+review
relates to:
auth0-ai-js:
auth0-lab/auth0-ai-js#285
auth0-ai-python:
auth0-lab/auth0-ai-python#51
auth0-ai-samples (python):
auth0-samples/auth0-ai-samples#15
auth0-ai-samples (JS):
auth0-samples/auth0-ai-samples#16
auth0-assistant0 (JS + python):
auth0-samples/auth0-assistant0#11
References
Testing
Checklist