Skip to content

Add credential scope update#23

Merged
herber merged 1 commit intomainfrom
add-credential-scope-update
Apr 17, 2026
Merged

Add credential scope update#23
herber merged 1 commit intomainfrom
add-credential-scope-update

Conversation

@herber
Copy link
Copy Markdown
Member

@herber herber commented Apr 15, 2026

Note

Medium Risk
Extends update payloads to include OAuth client_secret and scopes, which are security-sensitive fields; incorrect usage could overwrite credentials or send unintended values.

Overview
Enables updating provider auth credentials’ OAuth fields by extending the generated update methods to accept client_id, client_secret, and scopes, and serializing them into the PATCH request body.

Updates the corresponding generated *AuthCredentialsUpdateBody dataclasses/mappers (for both mt_2025_01_01_dashboard and mt_2026_01_01_magnetar) to include these new fields (with scopes defaulting to an empty list when deserializing).

Reviewed by Cursor Bugbot for commit 46c9b80. Bugbot is set up for automated code reviews on this repo. Configure here.

@herber herber self-assigned this Apr 15, 2026
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 46c9b80. Configure here.

@herber herber merged commit 3babec9 into main Apr 17, 2026
10 checks passed
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