Add test for dht servers when utp disabled

#289

Add establish outgoing connection expvar

Switch to goimports import sorting

Used to use sortimports, but it's old, and goimports seems to have an opinion now.

Add dial rate limiting

Fixes #273.

reverting usage of proxy for http requests

Fix timer leak in tracker announces when peers are wanted

Fixes #281.

Add go.sum file

Remove unused tracker.defaultClient

Fix TLS handshake failures with https trackers

Fixes #276

Always announce to trackers every interval, and announce earlier if short on peers

sortimports

Add a prefix to logonce logger output

Add comments to some ClientConfig fields

Revert "Make requestStrategy 3 the default"

This reverts commit 60dd0eed538361490cbf29a1734e6cd8d21f794f.

Improve tracker failure reason error

cmd/tracker-announce: Add a port flag

cmd/tracker-announce: Provide a http.Client for announces

cmd/tracker-announce: Was using empty AnnounceRequest

Ignore unused trailing bytes in HTTP tracker responses

Improve log message

Replace glycerine/goconvey with smartystreets/goconvey

It went missing, and some dependency wants it.

Merge branch 'master' into vgo-ci

Go module install/get make no sense

When Accept gives an error, don't use the Conn

Use go run instead of godo

And install godo too

I think go get becomes go install for arbitrary commands

Use go modules

Update go.mod

Track received handshake encryption activity

Use the new firewall callback support in go-libutp

Law of Demeter Client.mu

bencode: Remove a lot of expensive allocations

Merge pull request #269 from JyBP/master

cmd/torrent: Listen to termination signals

attempt to close the client on signal

bencode: Avoid an extra getTags call

bencode: Cache struct fields

bencode.Marshal: Get rid of the intermediate buffer

bencode: Add benchmark for krpc.Msg

Set Torrent.pieces cap too

We know exactly how many we want, and don't want to use any more memory.

Add iplist/cmd/iplist

Fix iplist.Range.String, and a comment

Don't return if there's an error accepting

This happens if there's too many file descriptors, and left the client unresponsive.

Wake connections that could send a request for a deleted piece

I'm not sure this is entirely correct, but not doing it is probably less correct. It should help prevent stalls where writers aren't requesting because they're starved out of opportunities by other connections.

Switch pieceIndex back to an int

I suspect that interface conversions using packet iter are causing a lot of allocation. Either way, with the casting this adds, we should be able to change pieceIndex's type alias now with minimal code change.

bencode: Avoid Value.Interface call testing for big.Int

Was resulting in significant allocation.

Don't randomize chunk order for requestStrategy 3

This should also reduce a lot of allocation.

Reduce default handshake timeout to 3s

Calculate peer priority lazily for worse conn comparison

Fix doc comment

Fix connection.utp

Track metadata chunks read with its own Count

Rewrite piece data decoding and relax test

peer_protocol: Add a test for receiving overlong piece data

The chunk buffer pool decides the upper bound on chunk data len.

Send extended message before fast/bitfield

This is how Transmission does it.

Fix peer_protocol.Message.RequestSpec for Type Piece

Add a test that short pieces are decoded correctly

Comments and trivial tweaks

Add peer_protocol.Integer.Uint32

Add peer_protocol.RequestSpec

Change pieceIndex to peer_protocol.Integer

Move back to net.IP for extended handshake message IPs

Don't error on bencode dicts that can't be marshalled

Have seen metainfo announce-lists that are lists of dicts. Possibly ignoring type errors with a tag would be smarter but I'm undecided.

Support marshalling bencode strings into byte arrays

Decide where to set proxy for HTTP trackers

Remove old ClientConfig.setProxyURL, set this directly now through ClientConfig.TrackerHttpClient.Transport.Proxy (if your Client allows it).

Remove debug log

Use half open limit directly from Config

Make extended handshake a struct, and move a bunch of extended stuff into peer_protocol

Move PEX stuff into peer_protocol

Reinstate the reduce dial timeout and update some values

Fixes #263

bep40Priority: Include error in return

Add more worseConn comparisons

Should fix #264.

More sortimports

sortimports

Fix panic when failing to parse tracker URL

Comments and readability

Add torrent.InfoHash type alias

Expose handshake stuff in peer_protocol

fs/test.sh: Add sudo before happy path umount again

Change the duplicateRequestTimeout to 1s

Assume connections have half the download bandwidth than calculated

Use timers for duplicate requests

Nothing was triggering request updates when timeouts expired. This is the simplest fix, no performance considered.

Run fs/test.sh in a temporary directory and make it clean up

Merge messageTypes{Sent,Posted} into torrent expvar

Make requestStrategy 3 the default

Fixes #253.

Fix panic in connection.iterPendingPieces when the info isn't available

Triggered by connection.iterUnbiasedPieceRequestOrder calling Torrent.numPieces. We shouldn't be iterating pieces when there's no info anyway, so don't yield any. Mentioned in https://github.com/anacrolix/torrent/issues/253#issuecomment-400350386.

Missed deferring release of an exported status writer

Fix racy Torrent.wantConns check when adding connections

Add more context to an error

Rework status export in tests to allow count > 1 without a server for every test

Do requests synchronously, and don't request from hashing or queued pieces

Calculating the desired state was a nice idea, but too hard to debug. This way should also be faster.

Check that chunks we request aren't being hashed or queued for hash

Fix BenchmarkConnectionMainReadLoop

Rework stats for receiving chunks

Related to #253.

Remove premature initialization of Torrent.lastRequested

Drop connections that send chunks we shouldn't receive

Add requestStrategy 3, which duplicates requests only after a timeout

Possible solution for #253.

Write received chunks that we don't have

Chunks were discarded if they belong to unprioritized pieces, even if those pieces were previously prioritized. May be contributing to #253.

Retain interest if requests are outstanding and don't clear peer requests if they send not_interested

Apparent shortcomings found when trying new request algorithms.

Ensure 64-bit alignment of ConnStats fields

Fixes #262.