DGNum/gestioCOF
16
2
Fork 1
Code Issues 78 Pull requests 14 Projects Releases 12 Packages Wiki Activity

Fix la redirection lors d'un logout CAS #689

Merged
lstephan merged 0 commits from Aufinal/fix_cas_redirect into master 2019-11-28 16:08:51 +01:00
Conversation 6 Commits - Files changed -
lstephan commented 2019-11-27 13:26:51 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Le CAS de l'ENS est mal paramétré, et le paramètre GET de redirection est service et non url comme prévu dans la doc CAS (et donc dans le module django-cas-ng)...

Le problème est signalé au SPI, mais en attendant on le patch à la main pour des raisons de sécurité (notamment pour le login générique en K-Fêt).

Effet de bord : quand on se logout de CAS sans redirection, il n'y a pas de message de logout : ça faisait juste du clutter inutile à la connexion suivante...

Le CAS de l'ENS est mal paramétré, et le paramètre GET de redirection est `service` et non `url` comme prévu dans la doc CAS (et donc dans le module `django-cas-ng`)... Le problème est signalé au SPI, mais en attendant on le patch à la main pour des raisons de sécurité (notamment pour le login générique en K-Fêt). Effet de bord : quand on se logout de CAS sans redirection, il n'y a pas de message de logout : ça faisait juste du clutter inutile à la connexion suivante...
mpepin commented 2019-11-27 16:26:50 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

lgtm mais je ne peux pas tester, tu as vérifié que ça fonctionnait ?

lgtm mais je ne peux pas tester, tu as vérifié que ça fonctionnait ?
mpepin commented 2019-11-28 16:08:51 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
mpepin commented 2019-11-28 16:08:52 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit 94d5e0f0ac

mentioned in commit 94d5e0f0accd4f8da857bfc6c006beee472cd9a0
mpepin commented 2019-11-28 16:10:23 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit 4c7993f48f

mentioned in commit 4c7993f48f77d6f5b993041b72fcb0113d7f8a08
mpepin commented 2019-11-28 16:12:02 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !383

mentioned in merge request !383
areitz commented 2019-12-02 22:51:52 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Fixes #236

Fixes #236
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
devtype -- backend

Au programme : du Python, du Django...

  
devtype -- docs

   
devtype -- frontend

   
devtype -- user interface

Au programme : templating, JS, CSS...

  
difficulty -- easy

   
difficulty -- hard

Difficile et/ou risqué

  
difficulty -- normal

   
Doing

   
domain -- bda

Concerne l'application BdA

  
domain -- bds

   
domain -- cof

Concerne l'application COF

  
domain -- core

Dépendances externes, CI, linting...

  
domain -- kfet

Concerne l'application K-Fêt

  
Good first issue

   
priority -- high

Urgent quoi

  
priority -- low

   
priority -- medium

   
priority -- staff-wanted

L'équipe dirigeante du domaine en question souhaite cette fonctionnalité.

  
status -- development

En cours de développement

  
status -- discussion

Avant de coder, c'est bien de discuter

  
status -- need review

En attente de code review

  
status -- production

L'ajout/la modification est présente en production

  
status -- ready to merge

Plus qu'à merger !

  
status -- todo

Le développement devrait bientôt commencer

  
To Do

   
type -- bug

Signalement ou correction de bug

  
type -- hygiene

Tout ce qui amène à du code plus propre et un repo mieux organisé.

  
type -- improvement

Ajout/Demande d'évolution mineure d'une fonctionnalité existante

  
type -- new feature

Ajout/Demande d'une nouvelle fonctionnalité

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#689
No description provided.
Delete branch "Aufinal/fix_cas_redirect"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?