nytimes/library

Add site access options for increased viewing permission granularity

Ouverte

#7 ouverte le 19 mars 2019

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (0 personne assignée)JavaScript (148 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (1 180 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Examinez la validation actuelle du domaine dans server/userAuth.js et explorez comment l'étendre pour prendre en charge les adresses e mail individuelles et les motifs regex.
Stack technique
javascriptnodejs
Domaine
backendauthentication
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Node.jsOAuth
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60