stress of tracker doesn't work due to tc->minInterval

[mtolman]

Please correct me if I am wrong.

In tracker.c, starting at line 271. As I understand it, if the trorrent has many peers, the code is supposed to check if we have very few peers connected and override the standard scrape interval with much shorter values i.e. 10 seconds.

The bug is in the three lines like the following:

if( now > tc->dateOk + 1000 * MAX( 10, tc->minInterval ) )

Since the server changes tc->minInterval upon init, the override fails since tc->minInterval will pretty much always be larger than 10. It is 1800 seconds on one server I frequent. I just removed the MAX() and hard coded the times needed.

if( now > tc->dateOk + 10000 )

[charles]

I think you're correct, but I'm wondering if this code is ever called at all since there's probably not a tracker on the planet that doesn't have minimum announce intervals. I wonder if it wouldn't make more sense to just remove this section of the code.

[charles]

We already reannounce as frequently as min_interval allows, and can't go any faster than that without getting banned, so the code as-is has no purpose. If we change it to the patch, we're actually going around min_interval, which would get us banned. IMO the only thing to do is remove the code.

[livings124]

With this code removed we're no longer using the min interval, instead we just follow the regular interval even if the min interval is lower. That code worked when there was no minimum interval or the minimum interval was less than the regular announce interval. At the least we should consider announcing when we're below the regular announce interval and above the min interval in cases with low peers (what this field is essentially there for).

[livings124]

The original way was correct, and the change has been reverted.

[anonymous]

Milestone 0.81 deleted