Move a few methods to Torrent, and track how many DHT announces occur

I suspect that DHT announces may occur too regularly, or without a break for torrents that aren't able to get over the lower water pending peer mark.

Client.announceTorrentDHT -> Torrent.announceDHT

bencode.Encoder: Unembed the Writer and wrap it in a minimal interface

I won't remove the bufio.Buffer for Marshal now, it isn't a performance issue as far as I can tell, and I'd have to fake Flush().

bencode: Remove private types encoder and decoder

Instead the functionality is exposed directly on the public types.

bencode.Unmarshal: Remove unnecessary intermediate bufio.Reader

bencode: Unembed the Reader type, and minimize the interface surface

Want to remove some unnecessary allocations.

README: Improve the cmd/torrent doc

Merge pull request #84 from gdouezangrard/master

fix disabling of global bootstrap servers for announce when the node table is empty

fix bootstrap from global servers disabling

Reduce the impact of preferring earlier pieces

I think urgent pieces at the end of a torrent were getting fairly starved.

Improve the logic for which address to use for DHT

Tidy up the listen function

Add tests for #81

Merge pull request #81 from lovedboy/master

fix listenedAddr  bug when TCP or UDP bind dynamic port

add return when utpsock's err is not nil

Merge branch 'master' of https://github.com/lovedboy/torrent

listenedAddr fix when TCP or UDP bind dynamic port

Ban IPs that contribute to failed piece checks

Torrent.MetaInfo: don't crash if info isn't available yet

Update Torrent.wantPeersEvent when info is obtained

Handle tracker announce prepare errors

Update Torrent.wantPeersEvent in several places

Add Torrent.AddTrackers

cmd/torrent-metainfo-pprint: Add Announce and AnnounceList keys

tracker: Skip any net errors to third party trackers

Drop the lol://cheezburger tracker from the test torrent

Spams the logs too much.

Don't start tracker scrapers if trackers are disabled

Don't create an empty tracker URL if both AnnounceList and Announce are empty

Rework torrent tracker scraping

New scrapers are added anytime trackers are added to a torrent. In the future they will also be stopped as soon as they're removed. All trackers are concurrently scraped, the old-style of sticking to one tracker that works is abandoned for now.

Comment on tracker.AnnounceRequest.Event

tracker: Allow resolving announce URL host in advance, and passing the desired Host field for HTTP requests

Also did some clean-up of the announce code, the concept of a client is going away.

Fix the tracker method receivers and names

Move torrent tracker methods into torrent.go

Torrent.writeStatus: Include human readable torrent size

Break out KRPC stuff from dht into new package

dht.Server.query: Return nil Transaction if there's an error

dht: Stop processing packet if server closes after unmarshalling

dht: Don't contact nodes with an IP containing a leading octet of 0

Improve dht.Transaction.SetResponseHandler comment

Improve the note about chunk writing concurrency

Everytime I see memory spike I suspect this, and it isn't it.

Changes to missinggo resource package

Rename some storage funcs

Rename storage.I->Client

Test both resource and file backends to piece storage

Remove unused fields in testDownloadCancelParams

Don't log missing files during hashing

Stop returning errors from the 2 callstacks to connectionLoop where errors are expected

Return an error from Torrent.setMetadataSize

This should mean connections sending ridiculous metadata sizes get dropped.

Move requestPendingMetadata onto connection from Client

iplist/cmd/pack-blocklist use tagflag

Add piece-per-resource storage

Probably fix test failure

Due to some changes in the Torrent instantiation code, I think there's a race in this test to verify the piece before counting the pending chunks. The torrent isn't supposed to be complete to begin with, so give it a bogus data directory to ensure it.

Bit of a tidy in the tracker code

Torrent.AddPeers can't fail, so don't return an error

Fix race marshalling a bitfield after a Have has also been posted

Update package doc

In particular, ConfigDir has gone away.

Get rid of obsoleted Torrent.ceaseNetworking, and clean up Torrent.closing

Rework Client listeners

In particular, if the ListenAddr used a dynamic port ":0", and both TCP and uTP were enabled. If the TCP listen succeeded, and the uTP did not, the TCP listener was leaked, and another port number was not tried.

Remove unused Client.pruneTimer

Wasn't using ChunkSize from TorrentSpec

Lock now required around missinggo.Event variables

Rework setting of info bytes

Add some comments to file storage

Add Client.AddTorrentInfoHash, and some other tidying

Remove Client.bannedTorrents, which shouldn't have been in use anymore. Also fix Client.newTorrent.

Remove the vestiges of the expose tracker Client interface

dht: Fix race contacting starting addrs in Announce

Fix dht.Server.Announce never stopping if the starting addresses can't be contacted

Move some behaviour into Client.newConnection

missinggo.Event changed, connection.writeOptimizer changes

tracker: Skip DNS error in UDP test

cmd/torrent-infohash update for API changes

Remove the last of the "config dir" stuff

This just conflates the Client. It should be done orthogonally.

metainfo.InfoEx.Hash becomes a function, UpdateBytes is added, and Bytes used in marshalling

Fixes #80.

metainfo: Add HashBytes

metainfo: Improve doc

metainfo.Info.BuildFromFilePath contained unnecessary log, and didn't handle error correctly

cmd/torrent-create: Tidy up flags and remove redundant function calls

Issue #80

Move a bunch of Torrent methods from client.go

Rework handling of metadata data messages

There are some dubious logs probably resulting from poor ordering of some code for handling incoming metadata data messages.

Doc and logging improvements

cmd/torrent-metainfo-pprint: Add -files flag, and expose NumFiles and TotalLength in output

Doc metainfo.Info.BuildFromFilePath

cmd/torrent-metainfo-pprint: Also print InfoHash

Add metainfo.Hash.FromHexString

Remove metainfo.Builder, and issue #35 test for it

Builder is poorly designed, and issue #35 is poorly written. I don't want to support either of them.

Allow encoding private=0 in metainfo, which fixes test for not using InfoEx.Bytes when marshalling as a shortcut

Also fix "context" in reader_test.go

Ugh, "context" isn't in go 1.6

Add Reader.ReadContext

Allows cancelling reads etc. Torrents that get stuck can result in Reads that won't return until the torrent is dropped.

Some changes to metainfo.InfoEx and testutil

Fix crash when peers have sent undersized bitfields and we get the torrent info

Change the behaviour of Torrent.Length with when info isn't available

Improve some docs

Fix receiver pointeriness for some File methods

cmd/torrent: Add -addr flag back in

Merge pull request #79 from shoenig/normalize

refactor struct identifiers to follow conventional go names

refactor struct identifiers to follow conventional go names

Improve doc on Torrent.Drop

https://github.com/anacrolix/torrent/issues/73

cmd/torrent: Forget trying to expose the entire Client config struct to the command-line

cmd/torrent: Remove redundant _ "net/http/pprof" import

It's done by envpprof instead