Tying some loose ends

src/converse-core.js

@@ -373,7 +373,7 @@
         // Module-level functions
         // ----------------------
 
-        this.generateResource = () => `/converse.js-${Math.floor(Math.random()*139749825).toString()}`;
+        this.generateResource = () => `/converse.js-${Math.floor(Math.random()*139749528).toString()}`;
 
         this.sendCSI = function (stat) {
             /* Send out a Chat Status Notification (XEP-0352)

src/converse-mam.js

@@ -84,10 +84,8 @@
 
         const messages = [];
         const message_handler = _converse.connection.addHandler(function (message) {
-            if (options.groupchat) {
-                if (message.getAttribute('from') !== options['with']) { // eslint-disable-line dot-notation
-                    return true;
-                }
+            if (options.groupchat || message.getAttribute('from') !== options['with']) { // eslint-disable-line dot-notation
+                return true;
             }
             const result = message.querySelector('result');
             if (!_.isNull(result) && result.getAttribute('queryid') === queryid) {

src/converse-register.js

@@ -334,7 +334,7 @@
                         return;
                     }
                     form.querySelector('input[type=submit]').classList.add('hidden');
-                    this.fetchRegistrationForm(domain);
+                    this.fetchRegistrationForm(domain.trim());
                 },
 
                 fetchRegistrationForm (domain_name) {