Skip to content

Commit

Permalink
review comment
Browse files Browse the repository at this point in the history
  • Loading branch information
ettec committed Dec 17, 2024
1 parent 6edeaa2 commit 1c3d92f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions core/capabilities/remote/trigger_subscriber.go
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ func (s *triggerSubscriber) UnregisterTrigger(ctx context.Context, request commo
return nil
}

func (s *triggerSubscriber) responseChannelRegister(workflowID string) bool {
func (s *triggerSubscriber) isWorkflowRegistered(workflowID string) bool {
s.mu.RLock()
defer s.mu.RUnlock()
_, found := s.responseChannels[workflowID]
Expand Down Expand Up @@ -183,7 +183,7 @@ func (s *triggerSubscriber) Receive(_ context.Context, msg *types.MessageBody) {
meta.WorkflowIds = meta.WorkflowIds[:maxBatchedWorkflowIDs]
}
for _, workflowID := range meta.WorkflowIds {
registered := s.responseChannelRegister(workflowID)
registered := s.isWorkflowRegistered(workflowID)
if !registered {
s.lggr.Errorw("received message for unregistered workflow", "capabilityId", s.capInfo.ID, "workflowID", SanitizeLogString(workflowID), "sender", sender)
continue
Expand Down

0 comments on commit 1c3d92f

Please sign in to comment.