Don't remove source knows block after verification (#2168)

* Don't remove source knows block after verification * Changelog
paritytech · Mar 23, 2022 · 970ad39 · 970ad39
1 parent df9aa6e
commit 970ad39
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/bin/wasm-node/CHANGELOG.md b/bin/wasm-node/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### Fixed
+
+- Fix regression introduced in version 0.6.5 where we erroneously removed entries in the mapping of which peer knows which blocks, leading to failures to request data. ([#2168](https://github.com/paritytech/smoldot/pull/2168))
+
 ## 0.6.7 - 2022-03-22
 
 ### Changed

diff --git a/src/sync/all_forks.rs b/src/sync/all_forks.rs
@@ -1584,10 +1584,6 @@ impl<TBl, TRq, TSrc> HeaderVerify<TBl, TRq, TSrc> {
                 // Block is valid!
 
                 // Remove the block from `pending_blocks`.
-                self.parent.inner.blocks.remove_sources_known_block(
-                    self.block_to_verify.block_number,
-                    &self.block_to_verify.block_hash,
-                );
                 let pending_block = self.parent.inner.blocks.remove_unverified_block(
                     self.block_to_verify.block_number,
                     &self.block_to_verify.block_hash,