cerebrate-project
/
cerebrate
mirror of https://github.com/cerebrate-project/cerebrate
1
0
Fork 0
Code Issues Releases Wiki Activity

Added checks for KC being enabled (#106) 

Initialisation of Auth behaviour and enrollment of users in KC, was dependent on the presence of a `keycloak` section in config.json

Added checks to ensure KC is configured as "enabled", rather than just checking if a keycloak config section was present.
pull/107/head
DocArmoryTech 2022-08-31 13:20:28 +01:00 committed by GitHub
parent 0537d05f53
commit cf565484b9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Model/Table/UsersTable.php
View File

@ -57,9 +57,11 @@ class UsersTable extends AppTable
private function initAuthBehaviors() private function initAuthBehaviors()
{ {
if (!empty(Configure::read('keycloak'))) { if (!empty(Configure::read('keycloak'))) {
if (Configure::read('keycloak.enabled')) {
$this->addBehavior('AuthKeycloak'); $this->addBehavior('AuthKeycloak');
} }
} }
}
public function validationDefault(Validator $validator): Validator public function validationDefault(Validator $validator): Validator
{ {
@ -188,7 +190,9 @@ class UsersTable extends AppTable
public function enrollUserRouter($data): void public function enrollUserRouter($data): void
{ {
if (!empty(Configure::read('keycloak'))) { if (!empty(Configure::read('keycloak'))) {
if (Configure::read('keycloak.enabled')) {
$this->enrollUser($data); $this->enrollUser($data);
} }
} }
} }
}