fix(store): properly update heightSub height

celestiaorg / go-header

Go library with all the services needed to request, sync and store blockchain headers.

Apache License 2.0

17 stars 16 forks source link

fix(store): properly update heightSub height #205

Closed cristaloleg closed 2 days ago

cristaloleg commented 3 days ago

Overview

PR #186 introduced a bug which resulted in an incorrect height reported by heightSub. heightSub.Height should be updated only when all height below the current one were observed.

Fixes #201 (+ test from #197)

cristaloleg commented 2 days ago

See #207