Better text encoding

Author: Tom32i

package-lock.json

@@ -58,7 +58,6 @@ export default class Client extends EventEmitter {
      */
     onMessage(event) {
         const { name, data } = this.encoder.decode(event.data);
-
         this.emit(name, data);
     }
 

src/client/Client.js

@@ -15,13 +15,15 @@ export default class StringCodec extends Codec {
      * {@inheritdoc}
      */
     encode(buffer, offset, data) {
-        const view = new DataView(buffer, offset, this.getByteLength(data));
-        const { length } = (data || '');
+        const encoder = new TextEncoder();
+        const bytes = encoder.encode(data || '');
+        const length = bytes.length;
+        const view = new DataView(buffer, offset, length + 1);
 
         view.setUint8(0, length);
 
         for (var index = 0; index < length; index++) {
-            view.setUint16(1 + (index * 2), data[index].charCodeAt(0));
+            view.setUint8(index + 1, bytes[index]);
         }
     }
 
@@ -31,12 +33,9 @@ export default class StringCodec extends Codec {
     decode(buffer, offset) {
         const view = new DataView(buffer, offset);
         const length = view.getUint8(0);
-        const chars = new Array(length);
+        const bytes = buffer.slice(offset + 1, offset + 1 + length);
+        const decoder = new TextDecoder('utf-8');
 
-        for (var index = 0; index < length; index++) {
-            chars[index] = view.getUint16(1 + index * 2);
-        }
-
-        return String.fromCharCode(...chars);
+        return decoder.decode(bytes);
     }
 }