Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Authentification par CAS #17

Merged
merged 7 commits into from
Jan 3, 2014
Merged

Authentification par CAS #17

merged 7 commits into from
Jan 3, 2014

Conversation

mattgu74
Copy link
Member

@mattgu74 mattgu74 commented Jan 3, 2014

Voici une PR, qui permet l'authentification CAS.
Si l'utilisateur n'existe pas il est automatiquement créé.

Une fois cette PR mergé, il restera quelque "TODO" à régler, notamment:

  • Mettre dans un fichier de config, l'url du CAS et l'url de Ginger (actuellement c'est en dur)
  • Générer un password aléatoire plutot que le login de l'utilisateur
  • Mieux gérer l'url de "service" (mettre en session) pour permettre une authentification sur toutes les pages.
    Actuellement on ne peut que s'identifier en revenant sur la page d'accueil.

This was referenced Jan 3, 2014
@mattgu74 mattgu74 mentioned this pull request Jan 3, 2014
@mattgu74
Copy link
Member Author

mattgu74 commented Jan 3, 2014

Oui, il y'a plusieurs endroits dans le code ou j'ai mis # TODO : ... Mais on réglera ça dans d'autres PR, c'est pour ça que j'ai crée les issues pour que l'on n'oublie pas. (Faut donc d'abord relire le reste puis merge).

flo-sch added a commit that referenced this pull request Jan 3, 2014
Authentification par CAS
@flo-sch flo-sch merged commit 5d0ae1f into payutc:master Jan 3, 2014
@mattgu74 mattgu74 deleted the CAS2 branch January 4, 2014 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants