[Feature Request] Allow passing a post-authentication callable to OIDCAuth
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Fonctionnalité
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- authentication
Piste de recherche
Commencez par le constructeur OIDCAuth et le point d’entrée OIDCAuth.callback décrits dans la demande, puis examinez le comportement existant de OIDCAuth autour de la redirection finale. Définissez l’argument de jeton du callable optionnel, l’accès à la session et le moment de l’exécution, puis ajoutez une couverture montrant qu’il s’exécute avant la redirection et qu’il est ignoré lorsqu’il n’est pas défini.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The OIDCAuth class and group-based features added in 2.3.0 are great!
It would be useful for the OIDCAuth class to take a callable that is executed at the end of the OIDCAuth.callback method just before the redirect. This function would take the access token as single positional argument and have access to the session global, e.g.:
from dash import Dash
from dash_auth import OIDCAuth
from flask import session
def add_scopes_to_user(token):
if scope := token.get("scope"):
session["user"]["scopes"] = scope.split()
app = Dash(__name__)
auth = OIDCAuth(
app,
secret_key="aStaticSecretKey!",
post_auth_callback=add_scopes_to_user, # defaults to None
)
This feature would provide a lot of flexibility to app developers for different use cases:
- storage of additional fields from the token in the session (example above),
- add a user to the DB if not already existing,
- custom logging,
- etc.
I can submit a PR if this is a feature the maintainers would consider merging.
- Langage dominant
- Python
- Étoiles
- 133
- Forks
- 67
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
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 plotly/dash-auth
-
Difficulté 4/5 3-5 jours Accessibilité débutants 28/100
-
Issue when clientside callback is defined before public callbackPeut-être pris @Godisemo l’a pris il y a 646 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
-
Docs: installation instructions (including oidc)Peut-être pris @davidhuser l’a pris il y a 698 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 55/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
Toutes les issues de plotly/dash-auth
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
databricks/databricks-sql-python#1034 ·
Les mainteneurs répondent en général sous 1 jour
-
premises-missing
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
WingedGuardian/GENesis-AGI#4432 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
huggingface/skills#301 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
intuitem/ciso-assistant-community#5275 ·
Les mainteneurs répondent en général sous 1 jour