Determine Best Auth Solution
@rmorshea y travaille déjà.
Depuis le 16/7/2023.
Évaluation
Cette issue n'a pas encore été évaluée.
Description
Current Situation
This stems from discussion in https://github.com/idom-team/idom/discussions/768
As explained in https://github.com/phihos/idom-auth-example-sanic, the problem we need to solve is how to securely authenticate users inside an ReactPy single page application. Usually authentication is done via Cookie or Authorization header on each HTTP request. But after the websocket connection has been established, no future headers will be sent. However, there are some ways you could try to work around this:
-
The user could push some Javascript that sends a separate auth request to your auth API and then reloads the page to reestablish the websocket connection with new auth headers, but this is not an elegant solution. It defeats the purpose of ReactPy not having to write any Javascript and having a visible reload has a negative impact on the user-experience.
-
You could also render the login page traditionally and then redirect to a new page with embedded ReactPy . But then you already split your application into two parts: "pre-auth" with traditional server-side template rendering and "post-auth" with ReactPy. Keeping both parts consistent is probably not fun.
-
You can also do authentication inside the single page app and save the auth state via
use_state. But it will be gone as soon as a websocket disconnect happens. You can mitigate this by pushing some Javascript that sets a session cookie. But now there is a new problem: Session cookies should be set with theHttpOnlyflag to prevent XSS attacks from recovering the session cookie. This can not be done (or at least is difficult to do) with Javascript. So you might end up with a security flaw in your app. -
Since you have at least one full HTTP request-response cycle you can set a session cookie with a session ID on that response if the request does not already contain a cookie with a valid session ID. That ensures that the following request for the websocket connection always contains a session ID cookie. With
use_requestwe can extract the session ID and then the server can retrieve the session data. In that data we can look up the authentication state and let ReactPy display a login form or the actual content. We can later manipulate the session data to perform a login or logout. All without the need to set a further cookie or push Javascript - provided we implement a server-side session. A rough prototype for this has been implemented here based on work done in https://github.com/phihos/idom-auth-example-sanic.
Proposed Actions
Explore the viability of each option.
- Langage dominant
- Python
- Étoiles
- 8.1k
- Forks
- 332
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de reactive-python/reactpy
-
Outdated GIF in Docs Ouvertepriority-3-low type-docs
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
reactive-python/reactpy#1034 ·
-
priority-2-moderate type-revision
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
reactive-python/reactpy#1319 ·
-
priority-1-high
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
reactive-python/reactpy#1277 ·
-
priority-3-low
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
reactive-python/reactpy#1273 · 1 réaction ·
-
priority-2-moderate
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
reactive-python/reactpy#1272 · 2 réactions ·
Toutes les issues de reactive-python/reactpy
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
zostera/django-bootstrap4#894 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
use-agent-os/agent-os#3276 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
zephyrproject-rtos/zephyr#119726 ·
-
area/auth bug comp/agent P3 platform/discord type/security
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
NousResearch/hermes-agent#117848 ·