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

Consider moving init code from app constructor to app on-enable trigger #99

Open michielbdejong opened 10 months ago

michielbdejong commented 10 months ago

We have code in https://github.com/pondersource/mfazones/blob/8cfbdd8ee507f14043c4dd947649f8195381e68a/lib/AppInfo/Application.php#L81 which runs the first time an admin logs in to a newly created server. It feels like this code would belong better in a trigger from occ apps:enable mfazones.

But from https://github.com/nextcloud/server/blob/master/apps/workflowengine/lib/Check/FileSystemTags.php#L109 it seems that a logged-in user is required to create a workflow so this is probably impossible.

Still, documenting the thought here in a GitHub issue, in case we learn more about this. :)