From: Christoph Wurst <christoph@winzerhof-wurst.at>
Date: Tue, 11 Jun 2019 05:26:41 +0000 (+0200)
Subject: Fix missing login hook wih Apache/SAML
X-Git-Tag: v17.0.0beta1~326^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F15909%2Fhead;p=nextcloud-server.git

Fix missing login hook wih Apache/SAML

Without this patch the hook does not transport the information whether the login is
done with an app password or not. The suspicious login app requires the parameter
to function correctly, hence adding it will make suspicious login detection also possible
with SAML users.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
---

diff --git a/lib/private/legacy/user.php b/lib/private/legacy/user.php
index f1c23381987..175c8f2e5d6 100644
--- a/lib/private/legacy/user.php
+++ b/lib/private/legacy/user.php
@@ -181,7 +181,15 @@ class OC_User {
 				// completed before we can safely create the users folder.
 				// For example encryption needs to initialize the users keys first
 				// before we can create the user folder with the skeleton files
-				OC_Hook::emit("OC_User", "post_login", array("uid" => $uid, 'password' => ''));
+				OC_Hook::emit(
+					'OC_User',
+					'post_login',
+					[
+						'uid' => $uid,
+						'password' => '',
+						'isTokenLogin' => false,
+					]
+				);
 				//trigger creation of user home and /files folder
 				\OC::$server->getUserFolder($uid);
 			}