nodeSolidServer / nodeSolidServer/node-solid-server

Rethink authentication configuration

Aberta
#672 11 comentários 3 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

authentication in progress semver-major
Linguagem predominante
JavaScript
Estrelas
1.8k
Forks
308
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

Right now, the server supports two different auth modes, confusingly named auth: "tls" and auth: "oidc", even though that doesn't cover exactly what they do. The main problem is that this one setting gives the impression that both methods are mutually exclusive, whereas they are not.

Discussions such as #667, #671, https://github.com/solid/solid-auth-client/issues/44, https://github.com/solid/solid/issues/138, https://github.com/solid/oidc-auth-manager/issues/20 show that there is a need for more clarity, and a better configuration option.

This issue tracks general progress and will act as a tracker for multiple sub-issues.
Here are the steps we need to take in my opinion:

  • Have one single contract for the entire node-solid-server to track a) the WebID of the logged-in agent b) the method through which that agents has logged in
  • Document the above contract
  • Ensure that “ask for a client certificate on every resource (WebID-TLS)” can be enabled and disabled independently of other features
  • Ensure that “act as an OIDC identity provider” can be enabled and disabled independently of other features
  • Ensure that “log in with OIDC through username/password” can be enabled and disabled independently of other features
  • Ensure that “log in with OIDC through client certificates (WebID-TLS)” can be enabled and disabled independently of other features
  • Ensure that “accept bearer tokens” can be enabled and disabled independently of other features
  • Examine whether we want to always translate successful logins into an Express cookie-based session
  • Examine whether we want to allow turning on or off cookie-based sessions
  • Create a configuration section that allows to enable and disable all of the above options independently
  • Update the init generator to create the right config
  • Remove the old auth config section

Feedback welcome.

Guia de contribuição

Abrir o guia de contribuição

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.

Direção de pesquisa

Comece lendo as discussões #667, #671 e as issues de autenticação vinculadas para entender o design não resolvido. Em seguida, rastreie a configuração de auth existente e o init generator, ambos mencionados na issue. O trabalho estará concluído quando a seção antiga de auth for substituída por controles independentes e documentados para as opções listadas de login, token, certificado, identity provider e sessão por cookie.

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

Avaliação

Stack de tecnologia
express, javascript, node.js
Domínio
authentication, backend
Tipo de issue
Funcionalidade
Dificuldade
5/5
Tempo estimado
Mais de uma semana
Status de atividade
Estagnada
Clareza
Precisa de esclarecimento
Facilidade para iniciantes
20/100

Receba novas issues na sua caixa de entrada

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