appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

Published 20 hours ago • verified publisher icon appsmithorg

[Task]: Add Error Handling for Oauth authorize and refresh flows

Open sanveer-osahan opened this issue 1 year ago • 2 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

SubTasks

Currently, the Oauth authorize and refresh flows are assuming only happy flows. Basic error handling is missing. Due to this, if anything fails, the user is not aware of the errors while using Appsmith and all they get is the 401 : Unauthorized message while using authenticated apis

sanveer-osahan avatar Apr 17 '23 08:04 sanveer-osahan

@sanveer-osahan can I take this up ? can you tell what we are trying to achieve and How do I reproduce this issue?

Abhijeetmishr avatar May 04 '23 13:05 Abhijeetmishr

Hey @Abhijeetmishr, yes you may take this up. Please follow the guidelines for contributing.

To reproduce this, create an Authenticated API datasource where the token endpoint returns an error during authorization. You can see that the datasource will be created successfully while it shouldn't. The API created on this datasource will give 401 error without letting the user know what was the actual error throughout this flow.

sanveer-osahan avatar May 08 '23 11:05 sanveer-osahan