Kick only the worst connection when a piece fails a check

Add a note about dropping connections that we sent a HAVE for incomplete pieces

Make Torrent.conns a map

Can't remember if I've tried this before. But dropping arbitrary connections while iterating established conns will become much simpler.

Move a bunch of Client methods onto more appropriate types

tracker: Support the original http response peers format

Fixes #130

Don't accept pieces with bad indexes

Fixes #132

Rename connection.downloadedChunk -> receiveChunk

Make downloadedChunk a method of connection

Move Client.downloadedChunk to connection.go

TODO

gofmt simplify

metainfo: Remove unused import

feat: remove print BuildFromFilePath

removes the print of (path, err) in function BuildFromFilePath

Reader.Close: Prefer to lock Client in public method

Torrent.unpendPieces: Only update unpended pieces

Updating all pieces at once is no longer faster. (There was an optimization for this a while ago that no longer exists.)

When Reader position changes only update piece priorities that may be affected

Seems to significantly reduce CPU load reading from torrents with huge piece counts.

Ditch the use of generic slices.HeapInterface for finding the worst "bad" connection

It's become a bottleneck. Implement it directly using heap.Interface.

Comment on connection.updatePiecePriority

There's also Torrent.updatePiecePriority, and it's always confusing what the difference is.

storage: ClientImpl interface now includes Close

This is now necessary because bolt doesn't allow multiple DB handles, so the bolt piece completion DB instance must be at the client level. The new method is not currently used from the Client however, this maybe necessary to avoid leaks.

storage: Fix spelling mistake

storage: Only include sqlite3 piece completion implementation if cgo is enabled

Issue #124.

storage: Remove sqlite3 import from bolt implementation

#124

Trivial HTTP support for adding torrent files by (#126)

* Trivial HTTP support for adding torrent files by

Sometimes it's really usefull to do not download
torrent file locally but simply pass HTTP link
this patch adds such ability.

Signed-off-by: Igor Shishkin <me@teran.ru>
* Fix for closing http connection after use

Signed-off-by: Igor Shishkin <me@teran.ru>

Add bolt piece completion DB

This means it can be persistent without needing cgo. Fixes issues #115 and #124.

Move the implementation of connection.requestPiecePendingChunks out of Torrent

Reader.SetReadahead can change readahead pieces

Optimize piece priorities when reader position changes

Gives a decent boost to throughput and reduces a lot of CPU when reading very quickly from Reader.

Add TODO item

Fix not working import

Add download rate limiting

Fixes #121.

Make newConnection a method on Client

Split connection.rw into separate Reader and Writer

This will make it easier to write hooks for Read and Write separately.

Fix Config.UploadRateLimiter comment

Add really simple upload rate limiting and test

Improve code comments

Reorder the fields of struct Client

Update README.md

The build status badge is not helpful due to the aggressive testing and badge caching.

Default to debug mode in client testing config

Do chunk pooling at Torrent instead of connection level

Speed up mse.handshake.establishS

Remove possible slice allocation, and avoid reflect.

peer_protocol.Decoder.Decode: Avoid allocating another intermediate reader

On my system, BenchmarkConnectionMainReadLoop goes from 596 to 1311 MB/s.

fix unreadable id when started

fix dht-server cmd won't exit after Ctrl+C

Create missing intermediate directories in test

Fix Client.receiveSkeys

Fixes #114. Loop variables in Go are ****ed.

Tidy up TestMultipleTorrentsWithEncryption test

Add some more debug logging

Formatting fixes

Fixes for changes to metainfo.MetaInfo.UnmarshalInfo

Add error return value to metainfo.MetaInfo.UnmarshalInfo

Too many problems caused by bad info bytes in downstream projects.

Add a test case for #114 (#117)

Add Config.{Force,PreferNo}Encryption

Minor comments and test improvement

storage: Move sqlite3 import to appropriate file

Comment on metainfo.PieceKey

Add note in README about the mobile branch

cmd/torrent: Fix progress bar for zero-length torrents

Use sync.pool for decode buffer

Rename some of the unexported file storage types

Test empty files and zero piece length for both file and mmap storage backends

Create zero-length files in the file storage when the torrent storage is opened

Fixes #111.

Fix issue #112

Add test for issue #111 and #112

Add a benchmark for readMainLoop piece decoding

Move Client.connectionLoop to connection.mainReadLoop

Remove unused trackerTier and shuffleTier

Move TorrentSpec stuff into its own file

Comment typo (#110)

Relax the seeder torrent connection stats test

The values have changed due to the recent storage change, I'm not sure why, and I don't know if it's important.

Rework storage interfaces to make them simpler to implement

This allows lots of behaviour to be baked into the new Client, Torrent and Piece wrappers, rather than duplicating (badly) them in all the backend implementations.

storage.boltDB: Some comments, and use global completedValue

Improve test reliability on Linux and reduce log noise in tests

Extract chunkSize as a const

Pass seed option to Config for cmd/torrent (#108)

Pass seed option to Config for cmd/torrent

Uncomment the error handling in storage.boltDBPiece.ReadAt

Add boltdb storage backend

Comment on PiecePriorityNext

Rework Reader position changes affecting piece priorities

Torrent.updatePiecePriorities is always followed by Torrent.piecePriorityChanged, so move it into the former function. Also drop the separate mutex for Reader, and add a missing Reader.posChanged call.

Improve comments

Add a benchmark for an observed slow case with Torrent.updatePiecePriorities

Only parse the config listen addr if we have to

This fixes a broken benchmark

Add missing error return check in benchmark

Make bencode.Bytes work with pointer and nonpointer receivers

Remove the InfoEx type, and don't generate its infohash on the fly

Fixes #106.

bencode: More renames

Add bencode.Bytes

bencode: Rename a bunch of slug case stuff

Speed up connection.lastHelpful

Moving to reflection and interfaces has made it a bottleneck.

cmd/torrentfs: Ensure unmounting occurs for some errors

Restructured so other errors can be handled better too.

Fix some more races in the status writing

fs: fix isSubPath for top-level directories (#105)

Fix build error with missinggo.Sorted

Always run seeder and leecher on the same ports, to flex dht.Socket.CloseNow

Add some extra conditions to prevent network activity when a torrent is closed

From suggestions by @axet, this might be a thing sometimes.

Begin exposing all the magic that Client.WriteStatus has so people can parse Client state their own way

It wasn't possible to move Client.WriteStatus to an external package to verify it doesn't depend on same-package access to data because then it can't be used in the tests, and it's extremely useful there. So I've settled for not locking the Client, and trying to use all the public methods. It's a work in progress.

it is actually sha1 hash size

torrentfs: fix panic when testPeer is not specified (#103)

Merge pull request #101 from perses/master

dirwatch: fix panic when attemting to copy func into [20]byte

torrentfs: fix panic when testPeer is not specified