Skip to content

Commit

Permalink
fix bug in fixmsgize that makes it stop after the first batch of 10k …
Browse files Browse the repository at this point in the history
…messages

for issue #71 reported by naturalethic, thanks!

users upgrading from v0.0.6 to v0.0.7 could run into this. the release notes
have been updated with a link to the issue. the issue will stay open until at
least the next release.
  • Loading branch information
mjl- committed Oct 5, 2023
1 parent 3aa5026 commit c4324fd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ctl.go
Original file line number Diff line number Diff line change
Expand Up @@ -758,6 +758,7 @@ func servectlcmd(ctx context.Context, ctl *ctl, shutdown func()) {
q.SortAsc("ID")
return q.ForEach(func(m store.Message) error {
lastID = m.ID
n++

p := acc.MessagePath(m.ID)
st, err := os.Stat(p)
Expand Down Expand Up @@ -810,7 +811,6 @@ func servectlcmd(ctx context.Context, ctl *ctl, shutdown func()) {
return fmt.Errorf("marshal parsed message: %v", err)
}
total++
n++
if err := tx.Update(&m); err != nil {
return fmt.Errorf("update message: %v", err)
}
Expand Down

0 comments on commit c4324fd

Please sign in to comment.