From 43c180c247a82ed20b2bc9e2de493305f7f5a43b Mon Sep 17 00:00:00 2001
From: David Renshaw <david@sandstorm.io>
Date: Sun, 9 Oct 2016 18:55:51 -0400
Subject: Make sure identities resolve before sending them to activity().

---
 sandstorm.js | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

(limited to 'sandstorm.js')

diff --git a/sandstorm.js b/sandstorm.js
index dc4a0796..5a800b24 100644
--- a/sandstorm.js
+++ b/sandstorm.js
@@ -108,12 +108,17 @@ if (isSandstorm && Meteor.isServer) {
     Meteor.wrapAsync((done) => {
       return Promise.all(users.map((user) => {
         return httpBridge.getSavedIdentity(user.id).then((response) => {
-          return { identity: response.identity,
-                   mentioned: !!user.mentioned,
-                   subscribed: !!user.subscribed,
-                 };
-        }).catch(() => {
-          // Ignore identities that fail to restore. Probably they have lost access to the board.
+          // Call getProfile() to make sure that the identity successfully resolves.
+          // (In C++ we would instead call whenResolved() here.)
+          const identity = response.identity;
+          return identity.getProfile().then(() => {
+            return { identity,
+                     mentioned: !!user.mentioned,
+                     subscribed: !!user.subscribed,
+                   };
+          }).catch(() => {
+            // Ignore identities that fail to resolve. Probably they have lost access to the board.
+          });
         });
       })).then((maybeUsers) => {
         const users = maybeUsers.filter((u) => !!u);
-- 
cgit v1.2.3-1-g7c22