Opened 9 years ago
Last modified 6 years ago
#691 new defect
I2PSnark sends interested message when not interested
Reported by: | sponge | Owned by: | zzz |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | apps/i2psnark | Version: | 0.9.1 |
Keywords: | I2PSnark snark interested bittorrent torrent performance | Cc: | |
Parent Tickets: | Sensitive: | no |
Description
I2PSnark sends interested message when not interested, followed by a not interested message.
While this does not break protocol, it's a waste of I2P's bandwidth, especially if the other peer unchokes and re-chokes. It also can eat upload slots on some clients disrupting the swarm from spreading pieces fairly.
Subtickets
Change History (5)
comment:1 Changed 8 years ago by
Milestone: | 0.9.2 → 0.9.5 |
---|
comment:2 Changed 8 years ago by
I think this is related?
<sponge> it was a great wait, but I finally have a screenshot of the I2PSnark bug that I've been attempting to explain unsuccessfully to zzz for years!
<sponge> zzz: Please see http://vnmf4apo3mpxitki2nzqjx24cq4ykuiptpdmihgyecomcql5ttaa.b32.i2p/files/I2PSnark-protocol-BUG.png for a screenshot of the torrent protocol bug I have been unsuccessfully attempting to explain to you all these years.
<sponge> zzz: once you fix this problem, I2PSnark *should* become as fast at downloading as Robert.
<sponge> If you rather I make a ticket with a full explaination, please let me know so that I can hand you the needed info of what is going wrong.
comment:3 Changed 8 years ago by
What seems to be happening is that these messages are sent when checking the piece array at the wrong time.
In order to fix the problem, here is what needs to happen.
When snark:
Completes the initial handshake set initial state to uninterested.
OR
Gets a have piece message from the peer.
OR
After a got message piece message is sent:
1: Compare peer have to local have arrays.
2: if the state has changed to interested or uninterested, send the appropriate message.
Do not send either message at any other time.
comment:4 Changed 8 years ago by
Milestone: | 0.9.5 |
---|
comment:5 Changed 6 years ago by
Keywords: | snark torrent performance added |
---|
Would you please provide additional context? Timestamped sequence of messages, perhaps? I need to know what happened before and after the two messages you describe. I don't have enough info yet to track this down. Thanks.