DeployBoard / DeployBoard/deployboard-python

Implement Okta allowed_groups and allowed_domains

Aberta
#143 0 comentários 0 reações 0 responsáveis Ver no GitHub
enhancement
Linguagem predominante
Python
Estrelas
10
Forks
1
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

**Is your feature request related to a problem? Please describe.**
We can support additional security around Okta auth by implementing `allowed_groups` and `allowed_domains`.
This would restrict who can even log into the application, although personally, I don't think there is any reason to deny anyone in your organization access to DeployBoard, but that's not my call to make about your org, so we'd like to support the option at least.

**Describe the solution you'd like**
After the Okta Oauth2 workflow, we retrieve the user's info and check if they are in any of the `allowed_groups` in the config.

We additionally would check the Okta domain the request is coming from to verify the domain matches the `allowed_domains` list. I kinda think this one is a bit redundant, since we require you to put the Okta client and secret in the config, so that should always match the domain associated with that client right? We're planning to support it anyway.

**Describe alternatives you've considered**
None

**Additional context**
None

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

Comece rastreando o fluxo de trabalho do Okta OAuth2 pela recuperação das informações do usuário e pelo tratamento da configuração. Determine como allowed_groups e allowed_domains devem ser representados e verificados após a autenticação; considera-se concluído quando usuários fora dos grupos ou domínios configurados não conseguem fazer login, enquanto usuários permitidos podem prosseguir.

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

Avaliação

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

Receba novas issues na sua caixa de entrada

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