Merge pull request #26 from jaseemabid/master

Clean up tracker network code

Fix bdict encode, cleanup blist encode

This one took a long time to debug! Need more unit tests

Clean up tracker network code

- Made info_hash an attribute of meta info
- Refactored a `get` function
- Removed few more useless types
- Cleaned up log messages
- Remove unnecessary Url type
- Fix double urlEncode

TIL You should ideally always write total functions in Haskell.

Merge pull request #25 from abhiranjankumar00/master

Follow up #21. Applying fmap (alternative) on getters

Follow up #21. Applying fmap (alternative) on getters

Merge pull request #24 from abhiranjankumar00/master

Fixes #21 (creationDate now not empty)

Fixes 21

Switch to known torrent file

Move things around

- `infoHash` is part of metainfo, not tracker alone. Peers need this for
handshake

- Rename PeerResp to TrackerResponse and move to Tracker.hs

- Remove unwanted overhead like `type Port = Integer`

- Some more docs, haddock is showing them nicely

- Update tests

Meta info tests

Expose all MetaInfo constructors

Use moby_dick.torrent for testing

`pieces` field of the Debian ISO is too long to put in a source file and
would cause Emacs to hang too often.

Minor refactor maybeBstrToString

Makes the function total. Also, this looks a lot like fmap!

Merge pull request #20 from vu3rdd/12-announcelist-2

combine announce and announceList slots in the Metainfo record.

use maybeToList from Data.Maybe

combine announce and announceList slots in the Metainfo record.

This eliminates the unnecessary "getter" function getTrackers
and simplifies the code a bit. Still a work in progress.

remove an unwanted import which creeped in during merge

remove unnecessary parenthesis (hlint suggestion)

support for announce-list.

Some torrent files use announce-lists but then store list of lists,
each inner list containing just one string instead of list of strings.
(eg: data/ubuntu-14.10-desktop-amd64.iso.torrent stores it this way)

Handle error cases from the tracker

Tracker might reply that the requested torrent is unavailable. Sample
response at `data/debian-7.8.0-amd64-CD-1.iso.error`.

Renamed `getPeerResponse` to `mkPeerResp` and changed it singture to
match other `mk*` functions like `mkMetaInfo`.

Added test cases

New checks made the code one more level deep. Will need a refactor with
the Either Monad.

Add directory & tasty as a library dependencies

Merge pull request #19 from jaseemabid/master

Cleanup a few bad types

Cleanup a few bad types

1. Change type InfoDict from `Map BVal BVal` to `Map String BVal`

The keys are human readable keys like 'name', 'announce' etc and String
is the natural fit. This change also gets rid of a lot of packing
unpacking business.

Example:

    -infoHash :: Map BVal BVal -> ByteString
    -infoHash m = let info = m ! Bstr (pack "info")
    -             in (hash . pack . encode) info
    +infoHash :: InfoDict -> ByteString
    +infoHash m = hash . encode $ (m ! "info")

2. Change type of `encode` from `BVal -> String` to `BVal -> ByteString`

Encoded values might not be ASCII safe in the first place. Also, it
makes sense to be the inverse function of decode. `encode . decode` pair
might be a good candidate for Quickcheck.

Other changes are consequences of the type change. Fixed a bunch of
broken doctests in Bencode.

TIL haskell is kickass for refactoring!

add dependency on package "directory" to functorrent-test

error out if the input string is not a valid filepath

Merge pull request #15 from vu3rdd/14-hlint-0

Run HLint as a test suite.

FIlenames start with capital letters. hlint.hs -> HLint.hs

travis.yml: install "happy" which is a dependency for haskell-src-exts

Run HLint as a test suite.

Fix cabal dependencies

Fix tracker response test

Fix key error

Fix test which did file parsing on tracker response

Relates to #11

README.md: hopefully correct image link this time

README.md: bang for the image link

README.md: messed up the markdown link syntax

README -> README.md

README: add travis build/test status badge.

Merge pull request #12 from jaseemabid/master

Add tests to functorrent

Add travis config

Tests for decode

Rename getPeers -> peers

Remove duplicate import

Remove dependency on time package

Make functorrent a library and an executable

This is a prerequisite for testing, as well as code organization.

1. `$ cabal repl` will build and import Functorrent library.
2. Making module exports explicit

Merge pull request #10 from jaseemabid/master

Make network request return ByteString

Make network request return ByteString

Packing and unpacking curropted the binary data in several occations
when the network response was written to disk for caching/testing.

Add support for local caches. This aids in testing, and throttling
remote requests if the file is new enough.

Removes unwanted <$>

Merge pull request #7 from dfordivam/master

Implemented logger

Feedback from @jaseemabid, Fixes #6

Logger implementation code. Fixes #6. Logging to file remaining

Merge pull request #1 from vu3rdd/master

merge from vu3rdd

Merge pull request #5 from jaseemabid/master

Make `!` inline

Make `!` inline

Merge pull request #4 from jaseemabid/master

Absolute imports, removing qualified imports

Fix failing test

Absolute import everywhere, cleanup

Merge pull request #3 from jaseemabid/master

Derive `show` automatically for BVal

Derive `show` automatically for BVal

more refactoring: type synonym for info dictionary

more refactoring around infohash

refactor PeerID and associated functions.

Until now we were hashing peer id string to get a 20-byte hash. But
that's not needed. Peer ID is just a 20-byte string that needs to be
unique per client and sent to the tracker (and peers).

WIP: peer handshake

Merge pull request #2 from jaseemabid/master

A few lint fixes to get started

A few lint fixes to get started

- Making `GHC -Wall` and hlint happy.
- Renaming length to len, to not shadow prelude function
- Remove unused import

Update current status.

Handle the `left' parameter for the tracker request.

Tracker initial request includes a parameter called `left' to indicate
the size of the file left to be downloaded. When we contact the tracker
initially, we haven't downloaded anything yet. So, the left parameter
should be the length parameter. For now, we handle only single file
torrent files, so we just pick the length param from Metainfo and shove
it in into the tracker http request.

add a TODO item on test suite

cute hlint suggestion for transformng a function

fix changed made during sleep deprived state

Use stackage

incorporate hlint suggestions

fix broken links

project renamed to "functorrent"

misc changes to fix warning

TODO: implement complete/incomplete fields

refactoring

haskell style guide link and spec link

print peer ip and port numbers

update README with a note on tests

update readme with goals and todos

more refactoring

add some test torrent files

refactoring

talk to tracker and get the peer dict

convert to base16 (hex) before doing urlencode

WIP: tracker protocol

refactor urlEncode

urlencoding: ByteString -> String

first attempt (buggy) at urlencode

minor formatting

info hash calculation

encode with tests

WIP: impement benc encoding, needed for infohash computation

WIP: Johan Tibbel's haskell style guide compliance.

https://github.com/tibbe/haskell-style-guide/blob/master/haskell-style.md

WIP: tracker interactions

HTTP client library dependency

parse and print Metainfo

WIP: Main

typos and correct module namespacing

WIP: mkMetaInfo