[NEW] Settings to further customize GitLab OAuth

app/gitlab/lib/common.js

@@ -1,13 +1,15 @@
 import { Meteor } from 'meteor/meteor';
 import { Tracker } from 'meteor/tracker';
+import _ from 'underscore';
 
 import { settings } from '../../settings';
 import { CustomOAuth } from '../../custom-oauth';
 
 const config = {
 	serverURL: 'https://gitlab.com',
-	identityPath: '/api/v3/user',
+	identityPath: '/api/v4/user',
 	scope: 'read_user',
+	mergeUsers: false,
 	addAutopublishFields: {
 		forLoggedInUser: ['services.gitlab'],
 		forOtherUsers: ['services.gitlab.username'],
@@ -19,16 +21,37 @@ const Gitlab = new CustomOAuth('gitlab', config);
 
 if (Meteor.isServer) {
 	Meteor.startup(function() {
-		settings.get('API_Gitlab_URL', function(key, value) {
-			config.serverURL = value.trim().replace(/\/*$/, '');
+		const updateConfig = _.debounce(() => {
+			config.serverURL = settings.get('API_Gitlab_URL').trim().replace(/\/*$/, '') || config.serverURL;
+			config.identityPath = settings.get('Accounts_OAuth_Gitlab_identity_path') || config.identityPath;
+			config.mergeUsers = Boolean(settings.get('Accounts_OAuth_Gitlab_merge_users'));
 			Gitlab.configure(config);
-		});
+		}, 300);
+
+		settings.get('API_Gitlab_URL', updateConfig);
+		settings.get('Accounts_OAuth_Gitlab_identity_path', updateConfig);
+		settings.get('Accounts_OAuth_Gitlab_merge_users', updateConfig);
 	});
 } else {
 	Meteor.startup(function() {
 		Tracker.autorun(function() {
+			let anyChange = false;
 			if (settings.get('API_Gitlab_URL')) {
 				config.serverURL = settings.get('API_Gitlab_URL').trim().replace(/\/*$/, '');
+				anyChange = true;
+			}
+
+			if (settings.get('Accounts_OAuth_Gitlab_identity_path')) {
+				config.identityPath = settings.get('Accounts_OAuth_Gitlab_identity_path').trim() || config.identityPath;
+				anyChange = true;
+			}
+
+			if (settings.get('Accounts_OAuth_Gitlab_merge_users')) {
+				config.mergeUsers = true;
+				anyChange = true;
+			}
+
+			if (anyChange) {
 				Gitlab.configure(config);
 			}
 		});

app/gitlab/server/startup.js

@@ -11,6 +11,8 @@ settings.addGroup('OAuth', function() {
 		this.add('API_Gitlab_URL', '', { type: 'string', enableQuery, public: true, secret: true });
 		this.add('Accounts_OAuth_Gitlab_id', '', { type: 'string', enableQuery });
 		this.add('Accounts_OAuth_Gitlab_secret', '', { type: 'string', enableQuery, secret: true });
+		this.add('Accounts_OAuth_Gitlab_identity_path', '/api/v4/user', { type: 'string', public: true, enableQuery });
+		this.add('Accounts_OAuth_Gitlab_merge_users', false, { type: 'boolean', public: true, enableQuery });
 		this.add('Accounts_OAuth_Gitlab_callback_url', '_oauth/gitlab', { type: 'relativeUrl', readonly: true, force: true, enableQuery });
 	});
 });

packages/rocketchat-i18n/i18n/en.i18n.json

@@ -117,6 +117,8 @@
   "Accounts_OAuth_Gitlab": "OAuth Enabled",
   "Accounts_OAuth_Gitlab_callback_url": "GitLab Callback URL",
   "Accounts_OAuth_Gitlab_id": "GitLab Id",
+  "Accounts_OAuth_Gitlab_identity_path": "Identity Path",
+  "Accounts_OAuth_Gitlab_merge_users": "Merge Users",
   "Accounts_OAuth_Gitlab_secret": "Client Secret",
   "Accounts_OAuth_Google": "Google Login",
   "Accounts_OAuth_Google_callback_url": "Google Callback URL",

server/startup/migrations/index.js

@@ -145,4 +145,5 @@ import './v144';
 import './v145';
 import './v146';
 import './v147';
+import './v148';
 import './xrun';

server/startup/migrations/v148.js

@@ -0,0 +1,24 @@
+import { Migrations } from '../../../app/migrations/server';
+import { settings } from '../../../app/settings';
+import { Settings } from '../../../app/models';
+
+Migrations.add({
+	version: 148,
+	up() {
+		settings.get('Accounts_OAuth_Gitlab', function(key, value) {
+			if (value) {
+				const setting = Settings.findById('Accounts_OAuth_Gitlab_identity_path');
+				const newValue = '/api/v3/user';
+
+				if (setting) {
+					Settings.updateValueById('Accounts_OAuth_Gitlab_identity_path', newValue);
+				} else {
+					Settings.createWithIdAndValue('Accounts_OAuth_Gitlab_identity_path', newValue);
+				}
+			}
+		});
+	},
+	down() {
+		// Down migration does not apply in this case
+	},
+});