Old blocks are ignored when processed

[lsebrie]

If the blocks are not close enough by block number when are relayed, then the block will be ignored. If it was a useful block it will be obtained later

[rskops]

SonarQube analysis reported 13 issues

• 9 major
• 4 info

Top 10 issues

1. RskFactory.java#L107: Method has 8 parameters, which is greater than 7 authorized.
2. RskFactory.java#L116: This block of commented-out lines of code should be removed.
3. RskFactory.java#L144: Method has 19 parameters, which is greater than 7 authorized.
4. BlockSyncService.java#L1: Class co.rsk.net.BlockSyncService creates and initializes a collection but never reads or gains information from it
5. BlockSyncService.java#L99: Method co.rsk.net.BlockSyncService.processBlock(Block, MessageChannel, boolean) appears to call the same method on the same object redundantly
6. NodeBlockProcessor.java#L284: Method co.rsk.net.NodeBlockProcessor.processSkeletonRequest(MessageChannel, long, long) does not presize the allocation of a collection
7. NodeMessageHandler.java#L80: Constructor has 8 parameters, which is greater than 7 authorized.
8. NodeMessageHandler.java#L106: The Cyclomatic Complexity of this method "processMessage" is 18 which is greater than 10 authorized.
9. NodeMessageHandler.java#L346: Method co.rsk.net.NodeMessageHandler.relayBlock(MessageChannel, Block) appears to call the same method on the same object redundantly
10. RskFactory.java#L116: Complete the task associated to this TODO comment.