Session cookie grows indefinitely, results in CSRF Warning. about authlib HOT 3 OPEN

Also reporting the same issue. What I did to solve this in our integration (see gradio-app/gradio#8000) is to catch the MismatchingStateError when it happens and delete all legacy states from the session cookie.

try:
    oauth_info = await oauth.huggingface.authorize_access_token(request)
except MismatchingStateError:
    # Delete all keys that are related to the OAuth state
    for key in list(request.session.keys()):
        if key.startswith("_state_huggingface"):
            request.session.pop(key)

    # Redirect back to login
    return RedirectResponse(login_uri)

Thanks @frozturk when founding out the root cause and hope this snippet will help future users until this bug is fixed.

(Regarding how to fix things in authlib itself, I don't know what should be the best approach. A naive solution is to clear previous states when adding a new one but it might cause (minor) discrepancies if a user is working in several tabs. @lepture Happy to open a PR though if this solution looks fine for you.)

from authlib.

@Wauplin A PR is welcome. Thank you.

from authlib.

@lepture I just opened #644. Please let me know what you think.

from authlib.