Add some tests verifying that writing the blocks for a Piece
with 3 blocks, in every permutation...

DiskManager now uses 'ITorrentData' to retrieve the list
of files, the torrent size and the piec...

This includes adding the capability to test the ratelimiting
code by faking how fast time is pro...

Add some APIs so we can check the cache hits and misses from
the MemoryWriter.

The DiskManager now stores/uses an IPieceWriter rather than
the old PieceWriter abstract class. ...

It's a bit nicer to have an extension method 'InvokeAsync' as it
is more natural. It allows usin...

The Dht engine is something we can Announce to, in order to inform
other peers we are downloadin...

PeerExchange messages would never be sent, or processed, for
private torrents. However to be a l...

Just makes sure that external peers cannot be added
to a private torrent using a bad ILocalPeerL...

The DhtEngine is registered at the ClientEngine, so the ClientEngine
should be the only one addi...

Also we need to call the new method when adding peers from
trackers!

Covers fix in a863bba8a202dbb3033357f4eda8ad0b712e454f

Some small renaming and better handling of default values.

Take advantage of the ability to await the hash checking code
to simplify concerns.

HashingMode...

Use autoprops. Rely on the base implementation of HandlePeerConnected
to do the right thing when...

Use auto props, use spaces instead of tabs, rely on the base
implementation of 'HandlePeerConnec...

Only one place in the code created this before the TorrentManager
was known. It is trivial to de...

Two different connections to the same ip/port/peerid should be treated
as distinct sessions as i...

StartAsync should signal that it is 'Complete' as soon as the mode
transitions to anything that ...

The PeerConnected/PeerDisconnected events expose the PeerId which
has Connected/Disconnected. Th...

As stopping a torrentmanager boils down to waiting for a few
Tasks to complete, we can simplify ...

This has been reworked to provide better insight into why
an outgoing connection attempt may hav...

This is consistent with the rest of the codebase and indicates that
these are just the range of ...

If the project has not been restored and something directly, or
indirectly, results in the custo...

Now releases can be independently re-created in a bit-identical
way. It still relies on having t...

The task returned by this method will transition to the
'Completed' state only when the Mode has...

We still default to using IPAddress/Port as some bittorrent
clients randomise their peerid, maki...

Also add a lot of code comments to explain the properties/methods.

The underlying SpeedMonitor is threadsafe, so there's no
need to add additional locking inside R...

The infohash should just be a 20byte BEncodedString. Only the
original http scrape request needs...

I wanted to clean up some naming collisions between namespaces.

In the MonoTorrent.Tracker name...

A bad blocking Wait or .Result could hang the loop, so ensure
that nothing deadlocks for both th...

We don't need a custom collection implementation.

This is just code hygiene, these are never null at runtime.

These ones were right after an 'await' so they are guaranteed to
succeed every time, but for cod...

We don't need to do a blocking .Result anymore as the containing
method (StartAsync) is now async!

Blocking is bad in general, so remove blocking calls. Also
add a ConfigureAwait false to reduce ...

If we used to have a list of trackers and it was removed, ensure
we remove the 'announce-list' k...

The connection ID should be written in network byte order, which
is handled automatically by the...

Less code is better!

This needs to be treated as a BEncodedString and not a normal
.NET string

These should all be maintained as BEncodedStrings so the raw
bytes are never corrupted. The test...

The tracker response should be kept as it's original
BEncodedString instead of being converted t...

Nothing used DataOffset anymore as the data is always at the start
of the 'Data' byte array.

Ta...

[doc] Update the README with list of implemented BEPs

In the majority case we request blocks (more or less) in order
and so we can hash each block as ...

If the initial buffer is unencrypted we can return it straight away
if plain text connections ar...

This is one fewer places where the catch-all 'PeerId' is not
actually needed. This simplifies th...

If the test takes more than 5 seconds then let's dispose
the connection and allow the test to fa...

This can be arbitrary bytes, but must be exactly 20 bytes in length.
As such we should treat it ...

The first block when requesting a piece using the fast peer extensions
was silently discarded in...

They indicate that monotorrent is used, the version which is running
and then end in a random un...

Some tests were added to cover cases where different strings
contained characters which need esc...

MonoTorrentCollection is not really needed, so hide it so it
can be removed later. Also remove s...

It's easier to maintain if I don't expose the implementations of
all the listeners as public API...

Share more code for these common tasks :)

This was just to save some lines of code writing argument null
checkers.

Move things from MonoTorrent.Common -> MonoTorrent.

If it's a common type then it should be in ...

Users of the library don't need to interact with these at all.

The tests cover all the cases nowadays, so this shouldn't be needed.

Huzzah!

The original reason for this split is that OpenSUSE wanted to ship
a build of MonoTorrent which ...

The MagnetLink object is immutable. There's a static Parse method
which takes the raw string, an...

Expose them as IListener and use ListenerFactory to obtain
either a UDP or HTTP listener. This a...

Change IListener to ITrackerListener so I don't have two different
IListener interfaces in the l...

Expose the trackers as a readonly IList.

Document this as returning the Tracker which will be used for
Announce/Scrape requests (when no ...