From 0b1925404c89213e3912f83036e4966cc2469ced Mon Sep 17 00:00:00 2001
From: Chris Morison <cmorison@gmail.com>
Date: Wed, 18 Dec 2019 03:46:36 +1200
Subject: [PATCH] Make server onLogOut hook only fetch user if required #10469

---
 packages/accounts-base/accounts_server.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/packages/accounts-base/accounts_server.js b/packages/accounts-base/accounts_server.js
index 87b049af5a..8d584cec7c 100644
--- a/packages/accounts-base/accounts_server.js
+++ b/packages/accounts-base/accounts_server.js
@@ -188,8 +188,10 @@ export class AccountsServer extends AccountsCommon {
   };
 
   _successfulLogout(connection, userId) {
-    const user = userId && this.users.findOne(userId, {fields: this._options.defaultFieldSelector});
+    // don't fetch the user object unless there are some callbacks registered
+    let user;
     this._onLogoutHook.each(callback => {
+      if (!user && userId) user = this.users.findOne(userId, {fields: this._options.defaultFieldSelector});
       callback({ user, connection });
       return true;
     });