Refresh token error codes are not unified across platforms and providers

[henninghall]

Issue

When the refresh token expires (could be due to changing password, not being used for a while etc) the error message sent from the library is different depending on platform and provider. It would be very nice if this could be unified.

Example

Responses for Google and Azure, havn't tried the others in the table below.

catch(e) {
    e.code // ErrorCode in example below
    e.message // ErrorMessage in example below
}

	Android	iOS
Azure	ErrorCode: Failed to refresh token ErrorMessage: AADSTS50173: The provided grant has expired due to it being revoked ....	ErrorCode: RNAppAuth Error ErrorMessage: invalid_grant: AADSTS50173: The provided grant has expired due to it being revoked
Google	ErrorCode: Failed to refresh token ErrorMessage: Token has been expired or revoked.	ErrorCode: RNAppAuth Error ErrorMessage: invalid_grant: Token has been expired or revoked.

Suggested fix

The same error should return the same error code defined by the library. The error messages could be kept as they are.

[kadikraman]

That is an excellent point, thanks for bringing this up! This is what happens when bridging independently maintained libraries in two different languages 😅

I think the Error Codes could be updated to be the same, but the Error Message is thrown by the underlying native library, so I'd like to keep it as is. What do you think?

[henninghall]

Yes, I totally agree 👍

[kadikraman]

Hey @henninghall - does this look good to you? #378

[henninghall]

Made one comment, check it out, otherwise, it looks good to me. Should probably be noted very clearly in the release notes since people might be relying on these old error codes.

[kadikraman]

Yes, good point. Thanks for your help! 🙌