Child Item

Access token not beeing refreshed automatically

Omri1984

my application support old dropbox and the new one.

when creating the client i am creating it like this

client = new DropboxTeamClient(request.AccessToken, request.RefreshToken,request.Expires,ClientId,ClientSecret);

and then using it

in this way

first Way

var members = await client.Team.MembersListAsync().ConfigureAwait(false);

and this way

Second Way

var account = await client.AsMember(request.AsMember).Users.GetCurrentAccountAsync();
var clientNew= client.AsMember(request.AsMember).WithPathRoot(new PathRoot.Root(account.RootInfo.RootNamespaceId));

var folders = await spaceclient.Files.ListFolderAsync(folderId);

my issue is after 4 hours when the token is expired

When i calling the first way i am getting an error

Response status code does not indicate success: 400 (Bad Request).

at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
at Dropbox.Api.DropboxRequestHandler.<RefreshAccessToken>d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Dropbox.Api.DropboxRequestHandler.<CheckAndRefreshAccessToken>d__21.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Dropbox.Api.DropboxRequestHandler.<RequestJsonStringWithRetry>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Dropbox.Api.DropboxRequestHandler.<Dropbox-Api-Stone-ITransport-SendRpcRequestAsync>d__12`3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
at MaaS.Systems.Dropbox.Service.DropboxService.<GetUsersAsync>d__26.MoveNext() in C:\development\CloudSfer\Systems\MaaS.Systems.Dropbox\Service\DropboxService.cs:line 157
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
at MaaS.Systems.Dropbox.DropboxServiceExtender.<GetHierarchy>d__7.MoveNext() in C:\development\CloudSfer\Systems\MaaS.Systems.Dropbox\DropboxServiceExtender.cs:line 126
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at MaaS.Web.Controllers.Api.DropboxController.<TreeViewNodes>d__7.MoveNext() in C:\development\CloudSfer\MaaS.Web\Controllers\Api\DropboxController.cs:line 175

it is look like it is failing on refreshing token .

the problem is that this request is not exist in the new client

Team.MembersListAsync(). not exist in the new client

please advice 🙂

Any Update Please

maybe this method client.Team.MembersListAsync() not handling as it should with the refresh token

Find more posts tagged with

API

Abuse

Comments

Greg-DB

The first way looks correct, and that code is working for me when I supply valid credentials. Make sure you're supplying the correct and valid values. For example, make sure:

the refresh token is exactly as originally provided by Dropbox
the refresh token hasn't been revoked by the app or user
the client ID and secret, a.k.a., app key and secret, are exactly as originally provided by Dropbox
the client ID and secret, a.k.a., app key and secret, are for the same app as the refresh token

For the second way, it is expected that "clientNew" would not have the MembersListAsync method, since the MembersListAsync method is only available on DropboxTeamClient instances, not DropboxClient instances. The AsMember method returns a DropboxClient instance to operate on a specific member, not the whole team. The WithPathRoot method sets the root for the files calls made by the client. Neither of these control the refresh process.

Omri1984

ok, but if i am calling the first way

i am getting the error.

I am trying to understand.

after 4 hours i am creating the dropbox team client with all the correct data(access,refresh,secret etc..).

but getting an error

can you try and reproduce it .

create a new token with 4 hours.

then wait 4 hours and then call the new dropbox team client with the token then you get before and call the client.Team.MembersListAsync()

i even try to call the refresh token method manually but this didn't work also

need your help 🙂

Greg-DB

Yes, I tried the code you shared, plugging in valid refresh token, etc. values, and it worked for me.

Please review the list in my previous message for reasons why that would fail for you. Check and update your values as necessary to make sure you're providing correct and valid refresh token, etc. credentials.

Omri1984

i an not understand and will need more assistance.

in my up i am always using the DropboxTeamClient creating it with all the necessary parameters

like this.

var client = new DropboxTeamClient(request.AccessToken, request.RefreshToken, request.Expires, ClientId, ClientSecret);

now after that i am checking if the client is in the new space or the old one

like this

var account = await client.AsMember(request.AsMember).Users.GetCurrentAccountAsync();
var spaceclient = client.AsMember(request.AsMember).WithPathRoot(new PathRoot.Root(account.RootInfo.RootNamespaceId));

if (account.RootInfo.IsTeam)
{

metadata = await spaceclient.Files.ListFolderAsync(folderId);
}
else
{
metadata = await client.AsMember(request.AsMember).Files.ListFolderAsync(folderId)
}

now sometimes i need to call this

client.Team.MembersListAsync().ConfigureAwait(false);

and befoe every call i am calling the

var client = new DropboxTeamClient(request.AccessToken, request.RefreshToken, request.Expires, ClientId, ClientSecret);

and I need it because that sometimes i am handing for the same client al the data content and all the user and groups as well

and my issue is that after the token is expire i am getting the error that was in the first message .

400 error

i am really need to get to the bottom of this and understand is completely

Thank you

Здравко

Hi @Omri1984,

Can you just print (or display somehow) content of "request.RefreshToken"? Is there something that looks like valid refresh token or not exactly? 🧐

Hope this gives direction.

Omri1984

something like this

ZEaFs***lgUAAAAAAAAAARUCp-9WwOK************************PIUr3P72**********Eg

does it make sense ?

Здравко

Hi again @Omri1984,

To be honest I expected your token empty, but it seems Ok, at least. Let's see is it really Ok or not actually. Execute following command in terminal:

curl https://api.dropbox.com/oauth2/token -d grant_type=refresh_token -d refresh_token=ZEaFs***lgUAAAAAAAAAARUCp-9WwOK************************PIUr3P72**********Eg -u 12APP_KEY3HERE45:abcAPP_SECRETdHEREefg

Place actual values on their corresponding places. You can use other standalone HTTP client, able to perform such a request, if curl is missing or inconvenient for your. Does the command completes successfully or gives back error (if error, what kind of error - post the result)?

Omri1984

will test it,

if i am asing a token with the business app or with the user app does the token is different between ?

Здравко

The token itself (both refresh and access) no (the above call is NOT affected). You have to take this in mind when actual API calls are performed (adding additional headers, whenever and if needed, on business account - as you are doing). That's it.

Здравко

@Omri1984 wrote:
...
and befoe every call i am calling the
var client = new DropboxTeamClient(request.AccessToken, request.RefreshToken, request.Expires, ClientId, ClientSecret);

and I need it because that sometimes i am handing for the same client al the data content and all the user and groups as well
...

You don't need to do the above "before every call". Changing client representation is enough. Otherwise, you are slowing down your speed up to 2 times (on every regular API call one more refresh call). It's meaningless.

Greg-DB

@Omri1984 It sounds like there's an issue with the credentials you're supplying, so trying the call outside of the SDK as Здравко suggested should be a useful test.

Omri1984

I added some logs to check my cred during the running .

I will update here, it holiday now so in a few days ,

thanks for the help.

by the way.

if create a new DropboxTeamClient(request.AccessToken, request.RefreshToken,request.Expires,ClientId,ClientSecret);

can I call those methods

}

await client.Team.MembersListAsync()

await client.AsMember(request.AsMember).Files.ListFolderAsync(folderId)

and another question ,When using the .Net SDK when is the refresh token is happening?

does it happen for every method ?

does it happen when creating the client (new DropboxTeamClient(request.AccessToken, request.RefreshToken,request.Expires,ClientId,ClientSecret);)

Thank you

Здравко

@Omri1984 wrote:
...
by the way.

if create a new DropboxTeamClient(request.AccessToken, request.RefreshToken,request.Expires,ClientId,ClientSecret);

can I call those methods

var account = await client.AsMember(request.AsMember).Users.GetCurrentAccountAsync();
...

Hi @Omri1984,

Overall idea is correct (only representation change 😉); you may need to check your syntax, etc in your end variant. Additional optimization can be done as "cache" your 'representation' , if in particular limited place multiple calls need to be performed sequentially (less objects construct/destruct -> will left one representation construct at beginning and one destruct at the end).

@Omri1984 wrote:
...
and another question ,When using the .Net SDK when is the refresh token is happening?
does it happen for every method ?
...

Yes and No. On every method call, SDK (all of them - not only .Net) check if current access token exists and its validity. That's what's done mandatory. If there is not available access token or it's going to expire (or already expired), refresh gets performed (which means once on every 4 hours roughly, if you use your code continuously).

@Omri1984 wrote:
...
does it happen when creating the client (new DropboxTeamClient(request.AccessToken, request.RefreshToken,request.Expires,ClientId,ClientSecret);)
...

No, it doesn't happen on client object creation time, but on methods execution (as I mentioned above). Warning: on client object creation, the state is reset, which means almost mandatory first call will need to refresh the token! As mentioned something meaningless. That's why it's not good idea to create client object continuously for every call, cosa the call gonna be forced to refresh (not the object construction itself). ".Net" SDK implementation doesn't provide currently some way to keep current state (like Java SDK or Python SDK). That's why the last state can't be restored (without some hacks/patches). Seems like a design bug. Let's hope this bug will be fixed.

Hope it's a bit more clear now.