fix: return a rejected promise when inner token is null or undefined

[erezrokah]

Related to decaporg/decap-cms#941, specifically decaporg/decap-cms#941 (comment)

When a call to _refreshToken inside the jwt methods fails, the inner token property is set to null here:

gotrue-js/src/user.js

Line 95 in 7f85187

this.clearSession();

That can happen when the refresh token is no longer valid as specified here netlify/netlify-identity-widget#142 (comment)

Subsequence calls to jwt will fail when trying to access a null token, but won't return a rejected promise thus breaking catch clauses like these ones:

gotrue-js/src/user.js

Line 72 in 7f85187

return this._request("/logout", { method: "POST" })

https://github.com/netlify/netlify-identity-widget/blob/c104dd18bb3b87d2a66936e15f064d5659259686/src/state/store.js#L146

since jwt is called on each _request that makes error handling very problematic once a refresh operation fails.

The fix is to return a rejected promise if the token is null or undefined

[ehmicky]

Looks good!

[erezrokah]

🎉 This PR is included in version 0.9.26 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀