fix(p2p): update height in blocksync loop

[srene]

PR Standards

Opening a pull request should be able to meet the following requirements

--

PR naming convention: https://hackmd.io/@nZpxHZ0CT7O5ngTp0TP9mg/HJP_jrm7A

---

This PR fixes an issue when DA is downloading blocks faster than blocksync, which may cause blocksync keeps downloading old blocks already applied.

Close #1031

<-- Briefly describe the content of this pull request -->

For Author:

• Targeted PR against correct branch
• included the correct type prefix in the PR title
• Linked to Github issue with discussion and accepted design
• Targets only one github issue
• Wrote unit and integration tests
• All CI checks have passed
• Added relevant godoc comments

---

For Reviewer:

• confirmed the correct type prefix in the PR title
• Reviewers assigned
• confirmed all author checklist items have been addressed

---

After reviewer approval:

• In case targets main branch, PR should be squashed and merged.
• In case PR targets a release branch, PR should be rebased.

[mtsitrin]

why not setting it in the for loop definition?
also need to validate that h progress, otherwise we can be stuck in endless loop (in case the height doesn't progress)

[srene]

i think is fine, because the max func will cause that h is always progressing, it will just skip already applied heights, in case state.NextHeight() is updated in parallel