[Feature Request] Allow passing a post-authentication callable to OIDCAuth

Offen
  • Stimmt etwas nicht?
#151 3 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
45/100
Issue-Typ
Feature
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
authentication

Rechercherichtung

Beginne mit dem in der Anfrage beschriebenen OIDCAuth-Konstruktor und dem OIDCAuth.callback-Einstiegspunkt und untersuche das bestehende Verhalten von OIDCAuth rund um die abschließende Weiterleitung. Lege das Token-Argument, den Sitzungszugriff und den Ausführungszeitpunkt des optionalen Callables fest und füge Tests hinzu, die zeigen, dass es vor der Weiterleitung ausgeführt wird und übersprungen wird, wenn es nicht gesetzt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.

Vorherrschende Sprache
Python
Sterne
133
Forks
67
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus plotly/dash-auth

Alle Issues in plotly/dash-auth

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.