emliunix commented on PR #5733:
URL: https://github.com/apache/opendal/pull/5733#issuecomment-2724368125

   Thanks for the ehancements, especially the addition of signer so that I can 
delete my custom token refresh logic.
   
   The code looks expecting one of 1. access_token or 2. refresh_token + 
client_id + optional client_secret to work.
   
   I tested the **refresh_token + client_id** case and found that I have to add 
scope **"offline_access"** to get the next refresh_token, otherwise it will be 
a json deserialize error complaining misssing refresh_token.
   
   ```
   2025-03-14T11:08:06.537397Z ERROR request{method=PROPFIND uri=/zotero/ 
version=HTTP/1.1}: opendal::services: service=onedrive name= path=/ listed=0: 
List::next failed Unexpected (permanent) at List::next => deserialize json
   
   Context:
      service: onedrive
      path: /
      listed: 0
   
   Source:
      missing field `refresh_token` at line 1 column 1627
   ```
   
   Tried following 2 workarounds but all failed:
   1. edit my my application (that of the client_id) to include offline_access
   <img width="830" alt="image" 
src="https://github.com/user-attachments/assets/99d117f6-dcea-472b-8ef5-e5e72ee739ac";
 />
   2. requesting offline_access on initial oauth workflow when obtaining 
refresh_token (+ access_token).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to