[FIX] Message_AllowedMaxSize fails for emoji sequences

package.json

@@ -136,6 +136,7 @@
 		"emailreplyparser": "^0.0.5",
 		"file-type": "^8.0.0",
 		"filesize": "^3.6.1",
+		"grapheme-splitter": "^1.0.2",
 		"gridfs-stream": "^1.1.1",
 		"he": "^1.1.1",
 		"highlight.js": "^9.12.0",

packages/rocketchat-lib/lib/MessageProperties.js

@@ -0,0 +1,14 @@
+import GraphemeSplitter from 'grapheme-splitter';
+
+const splitter = new GraphemeSplitter();
+
+export const messageProperties = ((message) => {
+	return {
+		length: splitter.countGraphemes(message)
+	};
+});
+
+// check for tests
+if (typeof RocketChat !== 'undefined') {
+	RocketChat.messageProperties = messageProperties;
+}

packages/rocketchat-lib/package.js

@@ -66,6 +66,7 @@ Package.onUse(function(api) {
 	api.addFiles('lib/RoomTypesCommon.js');
 	api.addFiles('lib/slashCommand.js');
 	api.addFiles('lib/Message.js');
+	api.addFiles('lib/MessageProperties.js');
 	api.addFiles('lib/messageBox.js');
 	api.addFiles('lib/MessageTypes.js');
 	api.addFiles('lib/templateVarHandler.js');

packages/rocketchat-lib/server/methods/sendMessage.js

@@ -29,10 +29,19 @@ Meteor.methods({
 			message.ts = new Date();
 		}
 
-		if (message.msg && message.msg.length > RocketChat.settings.get('Message_MaxAllowedSize')) {
-			throw new Meteor.Error('error-message-size-exceeded', 'Message size exceeds Message_MaxAllowedSize', {
-				method: 'sendMessage'
+		if (message.msg) {
+			const adjustedMessage = message.msg.replace(/:\w+:/gm, (match) => {
+				if (RocketChat.emoji.list[match] !== undefined) {
+					return ' ';
+				}
+				return match;
 			});
+
+			if (RocketChat.messageProperties(adjustedMessage).length > RocketChat.settings.get('Message_MaxAllowedSize')) {
+				throw new Meteor.Error('error-message-size-exceeded', 'Message size exceeds Message_MaxAllowedSize', {
+					method: 'sendMessage'
+				});
+			}
 		}
 
 		const user = RocketChat.models.Users.findOneById(Meteor.userId(), {

packages/rocketchat-lib/tests/server.tests.js

@@ -3,6 +3,7 @@ import 'babel-polyfill';
 import assert from 'assert';
 
 import PasswordPolicyClass from '../server/lib/PasswordPolicyClass';
+import {messageProperties} from '../lib/MessageProperties';
 
 describe('PasswordPolicyClass', () => {
 	describe('Default options', () => {
@@ -204,3 +205,20 @@ describe('PasswordPolicyClass', () => {
 		});
 	});
 });
+
+const messages = {
+	'Sample Message': 14,
+	'Sample 1 ⛳': 10,
+	'Sample 2 ❤': 10,
+	'Sample 3 ⛳❤⛳❤': 13
+};
+
+describe('Message Properties', () => {
+	describe('Check Message Length', () => {
+		Object.keys(messages).forEach((objectKey) => {
+			it('should treat emojis as single characters', () => {
+				assert.equal(messageProperties(objectKey).length, messages[objectKey]);
+			});
+		});
+	});
+});

packages/rocketchat-ui/client/lib/chatMessages.js

@@ -554,7 +554,13 @@ this.ChatMessages = class ChatMessages {
 	}
 
 	isMessageTooLong(message) {
-		return message && message.length > this.messageMaxSize;
+		const adjustedMessage = message.replace(/:\w+:/gm, (match) => {
+			if (RocketChat.emoji.list[match] !== undefined) {
+				return ' ';
+			}
+			return match;
+		});
+		return message && RocketChat.messageProperties(adjustedMessage).length > this.messageMaxSize;
 	}
 
 	isEmpty() {