Merge pull request #5944 from nextcloud/fix/take-sso-into-consideration-when-provisioning

Consider passwordless signins when provisioning accounts

1 files changed, 7 insertions, 1 deletions

diff --git a/lib/Http/Middleware/ProvisioningMiddleware.php b/lib/Http/Middleware/ProvisioningMiddleware.php
index 3acc1c8d9..8a99b57f8 100644
--- a/lib/Http/Middleware/ProvisioningMiddleware.php
+++ b/lib/Http/Middleware/ProvisioningMiddleware.php
@@ -70,9 +70,15 @@ class ProvisioningMiddleware extends Middleware {
 		}
 		try {
 			$this->provisioningManager->provisionSingleUser($configs, $user);
+			$password = $this->credentialStore->getLoginCredentials()->getPassword();
+			if ($password === null) {
+				// Nothing to update, might be passwordless signin
+				$this->logger->debug('No password set for ' . $user->getUID());
+				return;
+			}
 			$this->provisioningManager->updatePassword(
 				$user,
-				$this->credentialStore->getLoginCredentials()->getPassword()
+				$password
 			);
 		} catch (CredentialsUnavailableException | PasswordUnavailableException $e) {
 			// Nothing to update