getredash/redash

Allow name and email for user added by Remote User Auth to be different

Open

#1648 aperta il 2 mar 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Python (4236 fork)batch import
Backendenhancementhelp wanted

Metriche repository

Star
 (24.845 star)
Metriche merge PR
 (Merge medio 9g 5h) (3 PR mergiate in 30 g)

Descrizione

Issue Summary

I'm trying to use Active Directory LDAP through remote_user_auth module. Currently there is no way to set different redash user name from mail address. So now my redash user accounts have mail address for both name and mail. Also, if user logs in with just the username or username@domain, the mail address is not correct and the alarm mail won't work.

Technical details:

To solve this, I guess we have to add another environment variable besides REMOTE_USER_HEADER.

Does it make sense to you to add variable like REMOTE_MAIL_HEADER, and use it like

name = REMOTE_USER_HEADER
email = REMOTE_MAIL_HEADER || REMOTE_USER_HEADER
create_and_login_user(current_org, name, email)

instead of

email = REMOTE_USER_HEADER
create_and_login_user(current_org, email, email)
  • Redash Version:Current Head
  • Browser/OS:Chrome
  • How did you install Redash:Docker

Guida contributor

Direzione di ricerca
Esamina il modulo remote user auth in Redash, esplora come viene utilizzato REMOTE USER HEADER e implementa una nuova variabile d'ambiente REMOTE MAIL HEADER che consenta un'intestazione email separata, mantenendo il nome da REMOTE USER HEADER per garantire la retrocompatibilità.
Tech stack
python
Dominio
backendauthentication
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
PythonDocker
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75