nytimes/library

Add site access options for increased viewing permission granularity

Aperta

#7 aperta il 19 mar 2019

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (0 assegnatari)JavaScript (148 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (1180 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina l'attuale validazione del dominio in server/userAuth.js ed esplora come estenderla per supportare singoli indirizzi email e pattern regex.
Tech stack
javascriptnodejs
Dominio
backendauthentication
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsOAuth
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60