search

pondersource / nextcloud-mfa-awareness

Make Nextcloud aware of whether the current user is logged in with Multi-Factor Authentication
MIT License
0 stars 2 forks source link

Address review comments of PR 40235 #105

Open michielbdejong opened 7 months ago

michielbdejong commented 7 months ago

Specifically https://github.com/nextcloud/server/pull/40235#discussion_r1390886920 about using events, and two smaller issues.

michielbdejong commented 7 months ago

Need to think of a way to comply with https://github.com/nextcloud/server/pull/40235/files#r1390886920 Maybe have the MFA Checker app deal with this. The more thorough option would be triggering additional pull requests in GSS, user_saml, and https://github.com/nextcloud/server/tree/master/lib/private/Authentication/TwoFactorAuth