v1 refresh drive creds during perm sync

[evan-onyx]

Description

Fixes https://linear.app/danswer/issue/DAN-2012/refresh-drive-creds-during-perm-sync
When drive perm sync runs for a while, we sometimes get the following difficult-to-explain exception:

google.auth.exceptions.RefreshError: ('unauthorized_client: Client is unauthorized to retrieve access tokens using this method, or client not authorized for any of the scopes requested.', {'error': 'unauthorized_client', 'error_description': 'Client is unauthorized to retrieve access tokens using this method, or client not authorized for any of the scopes requested.'})

This can happen several hours into permission syncing despite the fact that drive tokens auto-refresh roughly every hour. So, presumably 1 or 2 refreshes succeed, then one fails.

How Has This Been Tested?

n/a

Backporting (check the box to trigger backport action)

Note: You have to check that the action passes, otherwise resolve the conflicts manually and tag the patches.

• This PR should be backported (make sure to check that the backport attempt succeeds)
• [Optional] Override Linear Check

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
internal-search	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 23, 2025 9:25pm

[greptile-apps]

PR Summary

This PR addresses a Google Drive token refresh issue during permission sync by introducing a RefreshableDriveObject wrapper and improved credential management.

• Added RefreshableDriveObject in resources.py to handle token refresh failures and retries during Drive API calls
• Modified get_permissions_by_ids in permission_retrieval.py to use RefreshableDriveObject instead of raw Resource
• Added _creds_dict storage in GoogleDriveConnector to maintain credential state for refresh operations
• Implemented _drive_connector_creds_getter in doc_sync.py to properly handle credential retrieval and refresh cycles
• Added error logging for missing owner emails and improved type hints for credential objects

_{4 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile}