Dumping and re-sending requests which occurred while refreshing about fresh

[vasilich6107]

I'm back with reproduction)

Here is a repo https://github.com/artflutter/fresh_repro

First of all let's start a demo server
go to jwt_server folder and run

npm install
npm run example

This will run a jwt server. There is also web interface for internal testing purposes http://localhost:12345/

Then launch flutter project.

There is 2 buttons Login and protected. Login - logs in user. Protected - launches request to token protected resource.
The tokens are managed via cookies but for the demo the actual way of token delivery to the server does not matter.

The initial token expiration is set to 1/6 of minute, 10 seconds.

In main.dart on line 79 you can tweak the refresh delay to simulate different scenarios.

[felangel]

@vasilich6107 sorry for the delay I've been really busy with other work this week. I will have a fix this weekend 👍

[felangel]

Hi @vasilich6107 👋
Thanks for opening an issue and for the positive feedback!

Are you able to share a sample app which reproduces the issue? I'm happy to take a closer look and update the package to account for this since it is definitely a place for improvement 😄

[felangel]

@vasilich6107 thanks so much! I'll have a look asap 😄

[vasilich6107]

Let me describe two scenarios) In next two messages

[vasilich6107]

First scenario - refresh and request repeat.

How to reproduce:

• open console to see the logs
• token refresh delay should be 0
• press login
• press protected several times until you see refresh operation message in console

There should be next message following refresh operation with flutter: {"message":"Some protected data"} cause after token refresh we need to resend failed request.
https://tppr.me/yRosQ

[vasilich6107]

Second scenario - multiple refresh request

How to reproduce

• make app hot restart to clear previous session if needed
• open console to see the logs
• token refresh delay should be 5000 (5 seconds)
• press login
• press protected "as fast as you can" to spawn multiple requests
• there will be a tons of refresh token requests in console

There should be only one refresh token requests. All other should be paused and resended after token successfully refreshed.
https://tppr.me/RBZr3

[vasilich6107]

Hi @felangel
Any news?

[marcos930807]

Any news on this, the static analysis on this package is broken

[felangel]

This should be addressed as part of #39

[vasilich6107]

Hi @felangel
This issue unfortunately is not fixed

I updated https://github.com/artflutter/fresh_repro to the latest versions of available libraries

It it still spams a lot of refresh requests while awaiting for token refresh

See the attached video

Screen.Recording.2022-05-17.at.18.13.12.mov

[vasilich6107]

@felangel any feedback?

[vasilich6107]

@felangel any news?

[felangel]

I think this should be resolved by #68. Will get that merged and published shortly

[vasilich6107]

Ok
Will check after merge