fit-ctu-discord / honza-botner

Bot for FIT CTU Discord
https://discord.fit.cvut.cz/
MIT License
15 stars 10 forks source link

Other schools verification #5

Closed ostorc closed 3 years ago

ostorc commented 4 years ago

As was suggested on discord talk, we should provide an option for other schools students to verify themselves and get by that access to the server, so mods doesn`t have to do it manually.

We should support at least these schools:

and maybe others.

jirkavrba commented 4 years ago

Maybe add non-cvut verification via email and match certain patterns?

*@fit.vut.br -> role:691907282019155989 atd

ostorc commented 4 years ago

That's an option. I would prefer to use OAuth if possible, but if not, that sounds like reasonable alternative.

sinus-x commented 4 years ago

Just a note: FEKT is part of VUT.

We do have this e-mail matching implemented in Rubbergoddess and it works flawlessly.

jirkavrba commented 4 years ago

That's an option. I would prefer to use OAuth if possible, but if not, that sounds like reasonable alternative.

We can use OAuth for CVUT sign-in, afaik VUT does have a SSO server, but there is no chance for us getting an app ID and app secret

sinus-x commented 4 years ago

We haven't been able to convince the IT department that we won't be threat to their system. There were also some GDPR issues they did not want to deal with.

pokorj54 commented 4 years ago

Is this really needed? We still need future students to use the server and I don't see a special feature for students from other universities. ¯\_(ツ)_/¯

sinus-x commented 4 years ago

This can automate the "I'm not from ČVUT but I want to be part of your server" comments mods have to deal with. E-mail verification for strangers is easy to implement and it saves a lot of time when the feature is in production environment.

tenhobi commented 4 years ago

Zira-like feature does automate this well enough. The server is PRIMARILY for the FIT CTU users, so wasting time on this feature while we don't have other stuff done is useless.