-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Add user backend registration to the bootstrap registration context #23370
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
Add user backend registration to the bootstrap registration context #23370
Conversation
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
| ]; | ||
| } | ||
|
|
||
| public function registerUserBackEnd(string $appId, string $class): void { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How come classes are passed as strings and not as interfaces? Shouldn't interfaces be enforced by type checks?
I understand this is already the case in the whole class, but I am not familiar with this concept.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The registration code is run for every single request. User back-ends are built most likely for those requests but other things are just loaded conditionally on demand (lazily) so less resources are wasted. Php allows us to specify the class name to load and we use the dependency injection container to create an instance when needed.
See #20573 for most of the original ideas and design decisions.
|
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 33974: failuresqliteShow full logmariadb10.1-php7.3
mariadb10.4-php7.4
mysql8.0-php7.4
mysql5.6-php7.3
postgres9-php7.3Show full logpostgres11-php7.4
acceptance-app-files-sharing
Show full log |
Because currently you have to query the internal user manager to do so.
Ref https://help.nextcloud.com/t/user-backend-api-overhaul/94896