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.

Merge branch 'dev'

Update accept limit parameters

Use scope timer in Torrent.writeChunk

moving upnp to a separate repository

Improve logging in a test

Disable accept rate limiting by default in tests

Fix multiLess.StrictNext

Fix build error with recent go version

Add some tests for net.Addr.Network values in various situations

bencode: Remove string allocation when parsing strings

cmd/torrent-metainfo-pprint: Use bufio input and enable envpprof

Log and handle error with bad tracker URLs

Move chunk expvars into torrent prefix

Use RLock for Client.WriteStatus

all: gofmt

Run standard gofmt command on project root.

- go version go1.10.3 darwin/amd64

Signed-off-by: ia <isaac.ardis@gmail.com>

Remove TestUTPRawConn

It's a bit racy and depends on flaky network. Something like this belongs in the utp package anyway.

Add connection.hasPreferredNetworkOver and friends

Fix some formatting of ConnStats fields

Add multiLess and use it for worseConn

Fix some testing code for Config->ClientConfig

Rework Torrent pending requests assertions

Should provide more info, and sooner for #249.

Allow dropping connections to the same peer ID

Necessary for a test that expects one connection to each other Client.

Torrent.Stats: RLock instead of Lock

Allow disabling accept limiting and modify some constants

Clear accept limits when adding a new torrent

Update external use of ClientConfig

Rename Config->ClientConfig and change how defaults work

This is a very breaking change.

Add lots of new expvars and perf timers

Remove old Client.dialTCP

Rate limit incoming IP prefixes

Lots of bad or dishonest incoming handshakes for unwanted torrents.

Use DialContext for tcp again

It got lost somewhere along the way.

Update dial expvars

Update some tests now that adding connections is more reliable

Remove excess newline from log call

Fix race condition in Client.ListenAddrs

Allow adding duplicate connections

I believe this fixes a long-standing race condition when dropping connections to the same peer ID where Clients assume the peer has added connections in the same order. Further commits may address duplicate connections further.

Fix Count usage in benchmark

Ensure ConnStats spew nicely

Fix build error with recent go version

Remove unused log import

Remove duplicateClientConns expvar

Update log call

Track ConnStats with atomics

Make download cancellable tests more reliable

Fix crash adding peers when Torrent is closed