Skip to content

🐛 fix issue with scoping for sheets - #20

Merged
guimorg merged 1 commit into
mainfrom
fix-scope-for-sheets
Apr 14, 2026
Merged

🐛 fix issue with scoping for sheets#20
guimorg merged 1 commit into
mainfrom
fix-scope-for-sheets

Conversation

@guimorg

@guimorg guimorg commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Moderate risk because it changes how Google auth credentials are prepared and authorized for Sheets/Drive access; incorrect scoping could break access for some credential types. Changes are covered by new unit tests for scoped vs non-scoped credentials behavior.

Overview
Fixes Google Sheets authentication by ensuring provided google-auth credentials have the required Sheets/Drive scopes before calling gspread.authorize, and emits a warning when credentials cannot be re-scoped.

Adds unit tests covering missing credentials, scoped credentials with/without Sheets scopes, and non-scoped credential warnings; includes a small test formatting tweak in BigQuery tests.

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

@github-actions github-actions Bot added the bug Something isn't working label Apr 14, 2026
@guimorg guimorg self-assigned this Apr 14, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 f9f8769. Configure here.

Comment thread gcpde/sheets.py
@guimorg
guimorg force-pushed the fix-scope-for-sheets branch from f9f8769 to 216257d Compare April 14, 2026 00:52
@guimorg
guimorg merged commit a65dc6a into main Apr 14, 2026
4 checks passed
@guimorg
guimorg deleted the fix-scope-for-sheets branch April 14, 2026 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

1 participant