[clement-chaneching]

Post-login actions after successful OIDC authentication.
For example, allows to pass custom attributes to the user session:

class MyOIDCAuth(OIDCAuth):
    def after_logged_in(self, user, idp, token):
        if user:
            user["params"] = value1
        return super().after_logged_in(user, idp, token)

For example, this would help to retrieve the "IDP" attribute so that we can manage permissions accordingly

[RenaudLN]

Thanks, a few small comments :)

[RenaudLN]

Thanks!

Just need to fix the linting, max 79 characters per line in this repo

[RenaudLN]

Actually, before approving, this ability should be documented somewhere. Can you add something in the class docstring to show the pattern with subclassing and overriding the after_logged_in method?

[RenaudLN]

👍

[RenaudLN]

@alexcjohnson would you have some time to give this one a look? It is handy to give devs the possibility to hook into the login process.

[alexcjohnson]

Looks reasonable to me. Probably good though to let either @gvwilson or @ndrezn take a look.

[clement-chaneching]

@alexcjohnson,
Could we merge this one and tag a new minor version?
This would help retrieving specific attributes after logging in from different Idps.
Thanks!

[clement-chaneching]

Chasing up on this one @alexcjohnson

[gvwilson]

Hi @clement-chaneching - my apologies for the delay at our end - we've been trying to get the new versions of plotly.js, plotly.py, and Dash wrapped up, and it has taken longer than expected. We have a meeting on January 7 to discuss contributions like this, so I've added a note to my calendar to get back to you on Wednesday January 8 about this. If you'd like to email me (greg.wilson@plot.ly) so that I have your address, I'd be happy to ping you that way as well. Thanks - @gvwilson

[gvwilson]

Hi again @clement-chaneching - apologies once more for the delay in merging this, and thank you again for your contribution.

[clement-chaneching]

Thanks! Nothing urgent, but let me know when a new minor tag will be released with this change.