Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CallbackError, keď login screen vyprší #456

Closed
luciajanikova opened this issue Jul 22, 2024 · 7 comments · Fixed by #482
Closed

CallbackError, keď login screen vyprší #456

luciajanikova opened this issue Jul 22, 2024 · 7 comments · Fixed by #482
Assignees

Comments

@luciajanikova
Copy link
Member

Keď sa používateľ prihlási do GovBox PRO cez google účet a hneď po prihlásení spraví krok späť na prihlásenie a znova klikne na výber účtu, tak vznikne OmniAuth::Strategies::OAuth2::CallbackError, viď screenshot z Rollbaru. Chceme vyriešiť, aby to pre používateľa nekončilo errorom.
image

@xhyrom
Copy link
Contributor

xhyrom commented Jul 22, 2024

related omniauth/omniauth-oauth2#95

@xhyrom
Copy link
Contributor

xhyrom commented Aug 8, 2024

V tomto prípade bude asi jediné riešenie si forknúť omniauth oauth2 a cherry picknúť. Skúsim sa s tým pohrať a ak to bude fungovať, dohodneme ďalej.

@xhyrom
Copy link
Contributor

xhyrom commented Aug 31, 2024

Podarilo sa mi to vyriešiť - xhyrom-utils/omniauth-oauth2@a6e7ede (postupoval som tak, ako písal gregnavis v issue hore). Musíme si to forknúť, keďže to oficiálne repo vyzerá neudržiavane.

@luciajanikova
Copy link
Member Author

@jsuchal pozries prosim?

@jsuchal
Copy link
Member

jsuchal commented Sep 5, 2024

@xhyrom @luciajanikova mam trosku averziu na forky a vseobecne chytanie security-related gemov - toto moze mat katastrofalne nasledky.

Ja mam pocit, ze tento error sa deje aj inokedy - napriklad ked mam login screen google dlho otvoreny a potom sa snazim prihlasit (asi session vyprsi). Mne pride, ze bezpecnejsie riesenie by bolo ten exception odchytit v rails alebo niekde a spravit logout alebo nic.

@xhyrom
Copy link
Contributor

xhyrom commented Sep 5, 2024

Tak dá sa to aj vyriešiť odstránením histórie a teda nebudeš môcť ísť back, ale odchytiť to môžme 🤷

@jsuchal
Copy link
Member

jsuchal commented Sep 5, 2024

Ja to chapem tak, ze ani teraz back nepojde, lebo to robi redirect aj tak hned spat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants