Skip to content

Commit

Permalink
Merge pull request #487 from icon-project/fix/solana-generate-message
Browse files Browse the repository at this point in the history
fix: ignore invalid event log
  • Loading branch information
sherpalden authored Feb 13, 2025
2 parents 1129cfb + fd1b498 commit 267a06d
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion relayer/chains/solana/provider.go
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,8 @@ func (p *Provider) GenerateMessages(ctx context.Context, fromHeight, toHeight ui

msgs, err := p.parseMessagesFromEvent(event)
if err != nil {
return nil, fmt.Errorf("failed to parse messages from event [%+v]: %w", event, err)
p.log.Warn("failed to parse messages from event", zap.Error(err))
continue
}
for _, msg := range msgs {
p.log.Info("Detected event log: ",
Expand Down
6 changes: 4 additions & 2 deletions relayer/relay.go
Original file line number Diff line number Diff line change
Expand Up @@ -676,8 +676,10 @@ func (r *Relayer) processAcknowledgementMsg(ctx context.Context, message *types.
messages, err = src.Provider.GenerateMessages(ctx, message.WrappedSourceHeight.Uint64(), message.WrappedSourceHeight.Uint64())
}
if err != nil {
r.log.Error("required message not found", zap.String("src", message.Src),
zap.Uint64("nid", message.MessageHeight))
r.log.Error("required message not found",
zap.String("src", message.Src),
zap.Error(err),
)
message.IncrementRetry()
message.ToggleProcessing()
return
Expand Down

0 comments on commit 267a06d

Please sign in to comment.