fix: signout bug when offline

[dehli]

Description of changes

If you call Auth.fetchAuthSession({ forceRefresh: true }) while offline and using the Authenticator, amplify-ui will trigger the logout flow. This fixes that bug.

Issue #6057, if available

Description of how you validated changes

Added a test and used patch-package.

This is the related amplify-js code:

https://github.com/aws-amplify/amplify-js/blob/0f5091780046b9556b98300c29fb970a0358bd70/packages/auth/src/providers/cognito/tokenProvider/TokenOrchestrator.ts#L173-L176

Checklist

• Have read the Pull Request Guidelines
• PR description included
• yarn test passes and tests are updated/added (when running locally they seem to run infinitely but I did add a test)
• PR title and commit messages follow conventional commit syntax
• If this change should result in a version bump, changeset added (This can be done after creating the PR.) This does not apply to changes made to docs, e2e, examples, or other private packages.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 001d09c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[dehli]

Bump. Been using this myself and it has addressed the sign-out bug we were previously experiencing.

[LukaASoban]

Bump - would love to see this fix go in as it is affecting all of our users

[calebpollman]

@dehli Thanks for opening this, fix looks straightforward. Will give it a full review in the next day or so

[calebpollman]

@dehli LGTM, just left a small suggestion. Thanks for cleaning up the typos

[dehli]

Thanks for feedback, @calebpollman! Added it as a follow-up commit. Let me know if you'd like me to squash them.

[calebpollman]

Thanks for feedback, @calebpollman! Added it as a follow-up commit. Let me know if you'd like me to squash them.

Thanks for addressing the feedback, no need to worry abt squashing - we'll handle that on our end

[calebpollman]

See #6367 for passing e2e tests