Release of the post login hook with OIDC

Aberta
#167 2 comentários 4 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
28/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
python
Domínio
authentication

Direção de pesquisa

Revise dash_auth/oidc_auth.py em torno do exemplo de after_logged_in na linha 273 e compare-o com o comportamento da versão 2.3.0 publicada. Rastreie como o callback OIDC invoca after_logged_in e, em seguida, inspecione o processo de release do repositório, se disponível. Considera-se concluído quando o hook documentado estiver incluído em um release publicado e o comportamento do seu callback tiver sido verificado.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

I was trying to implement some post login logic by following the instructions in the after_logged_in method from @clement-chaneching (https://github.com/plotly/dash-auth/blob/main/dash_auth/oidc_auth.py#L273), but could not get it to work.

 """
       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)
"""

I realized that this code was not released... Are there any plans for a new release integrating the latest changes to the package ?

Based on the code from 2.3.0, I ended up having to reset the server url rule for the OIDC callback in the __init__ of a MyOIDCAuth class, providing an overriden callback method (which then returns the overriden after_logged_in method). Without this my new after_logged_in method was never called (but it's possible I'm just confused about how inheritance works in this case).

Linguagem predominante
Python
Estrelas
133
Forks
67
Métricas de merge de PRs
Nenhum PR com merge em 30d

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de plotly/dash-auth

Todas as issues de plotly/dash-auth

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.