bito-code-review[bot] commented on PR #43944: URL: https://github.com/apache/superset/pull/43944#issuecomment-5563919082
<!-- Bito Reply --> The flagged issue describes a race condition where an older, failed request can clear the `started` state and display a generic error, potentially discarding a newer, successful refresh. This is a valid concern in asynchronous authentication flows where multiple requests might be in flight. To resolve this, you should ensure that the `started` state is only cleared if the failed request corresponds to the most recent token attempt. You can implement this by tracking the current request ID or timestamp and ignoring failures from stale requests. Would you like me to fetch all other comments on this PR to validate and implement fixes for them as well? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
