[fix #10157] Make sure onLogin callbacks are called during startup

packages/accounts-base/accounts_client.js

@@ -435,7 +435,22 @@ export class AccountsClient extends AccountsCommon {
     this._pageLoadLoginCallbacks.forEach(callback => callback(attemptInfo));
     this._pageLoadLoginCallbacks = [];
     this._pageLoadLoginAttemptInfo = attemptInfo;
-  };
+  }
+
+  // _startupCallback executes on onLogin callbacks
+  // at registration time if already logged in
+  // this can happen when new AccountsClient is created
+  // before callbacks are registered see #10157
+  _startupCallback(callback) {
+    // Are we already logged in?
+    if (this.connection._userId) {
+      // If already logged in before handler is registered, it's safe to
+      // assume type is a 'resume', so we execute the callback at the end
+      // of the queue so that Meteor.startup can complete before any
+      // embedded onLogin callbacks would execute.
+      Meteor.setTimeout(() => callback({ type: 'resume' }), 0);
+    }
+  }
 
   ///
   /// LOGIN TOKENS

packages/accounts-base/accounts_common.js

@@ -196,7 +196,10 @@ export class AccountsCommon {
    *                        as user details, connection information, etc.
    */
   onLogin(func) {
-    return this._onLoginHook.register(func);
+    let ret = this._onLoginHook.register(func);
+    // call the just registered callback if already logged in
+    this._startupCallback(ret.callback);
+    return ret;
   }
 
   /**
@@ -285,6 +288,9 @@ export class AccountsCommon {
     }
     return new Date() > (new Date(when) - minLifetimeMs);
   }
+
+  // No-op on the server, overridden on the client.
+  _startupCallback(callback) {}
 }
 
 // Note that Accounts is defined separately in accounts_client.js and

packages/accounts-base/package.js

@@ -1,6 +1,6 @@
 Package.describe({
   summary: "A user account system",
-  version: "1.4.4",
+  version: "1.4.5",
 });
 
 Package.onUse(api => {

packages/callback-hook/hook.js

@@ -77,6 +77,7 @@ export class Hook {
     this.callbacks[id] = callback;
 
     return {
+      callback,
       stop: () => {
         delete this.callbacks[id];
       }

packages/callback-hook/package.js

@@ -1,6 +1,6 @@
 Package.describe({
   summary: "Register callbacks on a hook",
-  version: '1.1.0'
+  version: '1.2.0'
 });
 
 Package.onUse(function (api) {