Make notification service username a config setting

shubhsherl · May 13, 2020 · e6e5ec8 · e6e5ec8
1 parent 577e65f
commit e6e5ec8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/app/api/server/v1/chat.js b/app/api/server/v1/chat.js
@@ -127,7 +127,7 @@ API.v1.addRoute('chat.postMessage', { authRequired: true }, {
 	post() {
 		// WIDECHAT only
 		let messageReturn;
-		if (this.user.username === 'viasat.notification.service') {
+		if (this.user.username === settings.get('Notification_Service_User_Username')) {
 			const { hostname_id } = this.requestParams();
 			if (!hostname_id) {
 				throw new Meteor.Error('error-hostname-id-not-provided', 'Body param "hostname_id" is required');

diff --git a/app/lib/server/startup/settings.js b/app/lib/server/startup/settings.js
@@ -925,6 +925,9 @@ settings.addGroup('General', function() {
 			public: true,
 			i18nDescription: 'Notifications_Always_Notify_Mobile_Description',
 		});
+		this.add('Notification_Service_User_Username', 'viasat.notification.service', {
+			type: 'string',
+		});
 	});
 	this.section('REST API', function() {
 		return this.add('API_User_Limit', 500, {