epicweb-dev / epic-stack

This is a Full Stack app starter with the foundational things setup and configured for you to hit the ground running on your next EPIC idea.
https://www.epicweb.dev/epic-stack
MIT License
4.32k stars 355 forks source link

fix: authenticator uses connectionSessionStorage for errors #755

Closed natac13 closed 3 months ago

natac13 commented 3 months ago

In utils/auth.server.ts the new Authenticator is called with the connectionSessionStorage. However, in the auth+/onboarding_.$provider.tsx route the code was trying to grab the sessionErrorKey from the authSession instean of the connnectionSession. This fixes that.

Test Plan

Simulate an error causd by the remix-auth authenticator failing when the GitHub (or other) social auth provider fails.

Checklist

Screenshots

fixes #750