From f23448be3340c56f6fae85b19a52aecf55e4753b Mon Sep 17 00:00:00 2001 From: guillaume Date: Thu, 22 Nov 2018 18:00:21 +0100 Subject: revert changes for patch authentication --- models/settings.js | 30 ++++++++++++++++++++++++++++++ models/users.js | 8 ++++---- 2 files changed, 34 insertions(+), 4 deletions(-) (limited to 'models') diff --git a/models/settings.js b/models/settings.js index 52212809..8d067c6d 100644 --- a/models/settings.js +++ b/models/settings.js @@ -239,5 +239,35 @@ if (Meteor.isServer) { cas: isCasEnabled(), }; }, + + getDefaultAuthenticationMethod() { + return process.env.DEFAULT_AUTHENTICATION_METHOD; + }, + + // TODO: patch error : did not check all arguments during call + logoutWithTimer(userId) { + if (process.env.LOGOUT_WITH_TIMER) { + Jobs.run('logOut', userId, { + in: { + days: process.env.LOGOUT_IN, + }, + on: { + hour: process.env.LOGOUT_ON_HOURS, + minute: process.env.LOGOUT_ON_MINUTES, + }, + priority: 1, + }); + } + }, + }); + + Jobs.register({ + logOut(userId) { + Meteor.users.update( + {_id: userId}, + {$set: {'services.resume.loginTokens': []}} + ); + this.success(); + }, }); } diff --git a/models/users.js b/models/users.js index 630f4703..2e879d94 100644 --- a/models/users.js +++ b/models/users.js @@ -520,10 +520,10 @@ if (Meteor.isServer) { } const disableRegistration = Settings.findOne().disableRegistration; - // If ldap, bypass the inviation code if the self registration isn't allowed. - // TODO : pay attention if ldap field in the user model change to another content ex : ldap field to connection_type - if (options.ldap || !disableRegistration) { - user.authenticationMethod = 'ldap'; + if (!disableRegistration) { + if (options.ldap) { + user.authenticationMethod = 'ldap'; + } return user; } -- cgit v1.2.3-1-g7c22