fix(feed): Properly get reactions from Posts endpoint (#1396)

* fix(feed): Properly get reactions from Posts endpoint * fix(feed): Unmarshal instead of Scan in handleExecuteReactPost in indexer
TERITORI · Nov 21, 2024 · 1ea33f7 · 1ea33f7
1 parent b9fd4e9
commit 1ea33f7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/go/internal/indexerhandler/feed.go b/go/internal/indexerhandler/feed.go
@@ -103,7 +103,10 @@ func (h *Handler) handleExecuteReactPost(_ *Message, execMsg *wasmtypes.MsgExecu
 	}
 
 	userReactions := make(map[string]interface{})
-	post.UserReactions.Scan(&userReactions)
+	if err := json.Unmarshal(post.UserReactions, &userReactions); err != nil {
+		h.logger.Error("failed to unmarshal UserReactions", zap.String("data", string(post.UserReactions)), zap.Error(err))
+	}
+
 	var users []networks.UserID
 	reactedUsers, found := userReactions[reactPost.Icon]
 	if found {

diff --git a/go/pkg/feed/service.go b/go/pkg/feed/service.go
@@ -171,8 +171,13 @@ func (s *FeedService) Posts(ctx context.Context, req *feedpb.PostsRequest) (*fee
 	posts := make([]*feedpb.Post, len(dbPostWithExtras))
 	for idx, dbPost := range dbPostWithExtras {
 		var reactions []*feedpb.Reaction
-		reactionsMap := map[string]interface{}{}
-		dbPost.UserReactions.Scan(&reactionsMap)
+
+		reactionsMap := make(map[string]interface{})
+		if err := json.Unmarshal(dbPost.UserReactions, &reactionsMap); err != nil {
+			s.conf.Logger.Error("failed to unmarshal UserReactions", zap.String("data", string(dbPost.UserReactions)), zap.Error(err))
+			continue
+		}
+
 		for icon, users := range reactionsMap {
 			ownState := false
 			if queryUserID != "" {