-
Notifications
You must be signed in to change notification settings - Fork 297
SDKs need to support credential-override #553
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
Comments
@sangee2004 this is implemented in all SDKs as of |
Tested with gptscript version
I dont have openai key set (as credential or env variable) and I set it using credential-override when executing chat script that has image-generation tool using claude provider. I am still prompted for entering openai keys which should not be the case. Following script was executed using -
Executing the same script from above using "--ui" option also results in user being prompted to enter openai key which should not be the case. Note - I am able to see credential-override feature work as expected in |
Fixes for credential-override to work with |
Tested with gptscript version - Able to see |
Tested with gptscript version - Able to see |
Tested with gptscript version - Able to use |
When executing scripts using Node SDK, credential overrides can be set using
Note - Setting |
AFAIK this is expected. As of now, I'm not aware of a requirement for the SDKs to have first-class support for |
We transitioned to using credentials for some of our tools, image-generation in particular, and when they are consumed programmatically through the APIs there is no way to programmatically set the credentials today.
The CLI supports both an environment variable and CLI argument approach. Both should be supported, but if one needs to be picked the CLI flag should be passable as a runtime option and the programming language can pull env vars and set them.
Today the environment variable is not picked for whatever reason, and there is no way to pass the CLI flag.
We need to provide a way to set this up.
The text was updated successfully, but these errors were encountered: