Conversation
Du coup on perd ça ? Perso de mon côté ça marche je suis pas certain de comprendre
La feature de base c'était pour permettre de lier / update son compte bar <=> compte tn.net Après si c'est le souhait, il faut modifier le front pour retirer la feature aussi |
|
Nan, normalement ça on garde. /auth/qr ça semblait être une méthode non achevée pour se connecter avec un qr code sans avoir à poser la carte. Turns out le code concerné (dans borneQr) mettait une valeur dans un cache qui était jamais querry et envoyait via un websocket un objet JSON qui était jamais traité. |
|
Ah ouais j'avais fait /auth/qr et /account/qr mais je me souviens même plus 🤣 C'est bien l'autre qu'on utilise depuis le front |
|
Les commits ne respectent pas la norme conventional commit, y'a moyen de juste rebase pour renomer ? |
florianmouro
force-pushed
the
feat/auth-rewrite
branch
from
2db9673 to
93329e0
Compare
3 participants
Summary
Refactor the OAuth integration in the project. Mostly here to isolate the current relevant code related to OAuth to rewrite later all authentication on the server side and allow easier integration of other OAuth provider.
It should fix broken authentication with broken redirects.
Removes /auth/qr which was not implemented fully and is not compatible with the refactor.
This PR is not supposed to change the behavior of the server compare to the current implementation, and so will make more sense after the refactor of the authentication middleware.
The project currently does not handle errors in a proper way, so this PR introduces the necessary structs to do it afterwards but does not use them.
Related issues :
None
Types of changes