From 4f852af4cec15d599f5594a550c2844ce44056df Mon Sep 17 00:00:00 2001 From: Bjoern Schiessle Date: Mon, 23 Apr 2018 11:40:47 +0200 Subject: [PATCH] don't auto provision the user on a global scale setup with global scale the authentication happens on the master node and then the user is forward to the node they are located. Therefore no user should be created on the master node after the authentication at the idp was successful Signed-off-by: Bjoern Schiessle --- lib/Controller/SAMLController.php | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/lib/Controller/SAMLController.php b/lib/Controller/SAMLController.php index 234be62..ace4fcd 100644 --- a/lib/Controller/SAMLController.php +++ b/lib/Controller/SAMLController.php @@ -98,6 +98,12 @@ class SAMLController extends Controller { * @throws NoUserFoundException */ private function autoprovisionIfPossible(array $auth) { + + // nothing to do here, in case of a global scale setup + if ($this->config->getSystemValue('gs.enabled', false)) { + return; + } + $prefix = $this->SAMLSettings->getPrefix(); $uidMapping = $this->config->getAppValue('user_saml', $prefix . 'general-uid_mapping'); if(isset($auth[$uidMapping])) {