-
Notifications
You must be signed in to change notification settings - Fork 1
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
Change auth to ULB's CAS/SAML #35
Comments
Would be nice ! I will have a look at the DocHub implementation. This would mean that we could include restrictions based on the faculty also ? |
I think we should make a proper database first (like not a Google spreadsheet) before accessing (and potentially storing) more students data. I may have someone willing to help us with that. |
I'm not sure, maybe
Yes! I also wonder if the IT staff can host a db for us at their on-prem. It would resolve some of the problems we keep having when self-hosting |
see their PR that implements it: DocHub-ULB/DocHub#192 |
here's a quick idea of a (flask?) website to link the accounts: https://codepen.io/lucasplacentino/full/QWYYVwB The registration flow would be: Users should be able to unlink their Discord by themselves (so it should remove their roles in all servers and remove their discord_username entry in the db) . |
See: Do we want to release it in this repo as a v2 or in a new repo as a new project? |
similarly to DocHub
(see https://github.com/DocHub-ULB/DocHub/blob/main/users/authBackend.py for example implementation)
I asked the ULB IT staff about it and we can get access for it, and it could include some information about the student (like name and surname, faculty, and student-status)
The text was updated successfully, but these errors were encountered: