nytimes/library

Add site access options for increased viewing permission granularity

Aberta

#7 aberto em 19 de mar. de 2019

Ver no GitHubGuia do colaborador
 (0 comentário) (1 reação) (0 responsável)JavaScript (148 forks)auto 404
enhancementgood first issue

Métricas do repositório

Stars
 (1.180 estrelas)
Métricas de merge de PR
 (Mesclagem média 8m) (1 fundiu PR em 30d)

Description

Problem Description

Currently, Library utilizes domains to control who can access the site. However, this prohibits access on a person-by-person basis and increases the amount of maintenance needed when different aliases of the same domain are often used (ex: student@u.northwestern.edu versus student@northwestern.edu).

Feature

To address these issues, Library should support:

  • individual email address access via an environment variable and OAuth configuration
  • the use of regex for domain matching

Additional Information

Library accesses a user's email domain via the user's passport session and checks it against the APPROVED_DOMAINS config var.

Guia do colaborador

Direção de pesquisa
Examine a validação atual de domínio em server/userAuth.js e explore como estendê la para suportar endereços de e mail individuais e padrões regex.
Pilha de tecnologia
javascriptnodejs
Domain
backendauthentication
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Node.jsOAuth
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60