-
Notifications
You must be signed in to change notification settings - Fork 180
Config changes for custom scopes support #1166
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
b50398b to
4fb6bd0
Compare
4fb6bd0 to
d6a9118
Compare
e9d4523 to
15d443c
Compare
15d443c to
73d2419
Compare
|
If integration tests don't run automatically, an authorized user can run them manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
| ("sql, offline_access", None), | ||
| ("sql, offline_access", '{"type": "databricks_resource"}'), | ||
| ("sql", None), | ||
| ("sql offline_access all-apis", None), | ||
| ("sql, offline_access, all-apis", None), |
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.
This is probably a breaking change, to avoid making it we will have to also allow for whitespace as a delimiter as runtime oauth already uses that format.
Summary
scopesis empty.How is this tested?
disable_oauth_refresh_tokenis set when environment variableDATABRICKS_DISABLE_OAUTH_REFRESH_TOKENis set.NO_CHANGELOG=true