Js-ipfs: WIP - Awesome Relay Endeavour

Created on 16 Apr 2017  路  5Comments  路  Source: ipfs/js-ipfs

We're getting close to completing the JS circuit implementation, this issue is to track further progress. Main work is being done here - https://github.com/libp2p/js-libp2p-circuit/pull/9~~ https://github.com/libp2p/js-libp2p-circuit/pull/14.

Assumptions (needs validation)

  1. All addresses are encapsulated with p2p-circuit by default to make them dialable over circuit relay
  2. If an explicit relay address is present don't add defaults, use that as the address instead
  3. Out of all available transports relay is dialed last

Outstanding work (this will be kept up to date with specific tasks as they come up):

  • [x] Complete the relay spec https://github.com/libp2p/specs/pull/15
  • [x] Verify and complete integration points in js-ipfs and libp2p

    • [x] Finalize integrating with libp2p

    • [x] Finalize integrating with js-ipfs

  • [ ] More test coverage and integration tests (WIP)

    • this is the latest coverage report for unit tests, lots more is needed but we're getting there

=============================== Coverage summary ===============================
Statements   : 62.12% ( 264/425 )
Branches     : 50.74% ( 69/136 )
Functions    : 50% ( 13/26 )
Lines        : 62.2% ( 255/410 )
================================================================================
  • [x] Cleanup implementation

    • [x] cleanup pull stream read/write logic

    • [x] cleanup relay.js

    • [x] cleanup listener.js

Outstanding PRs:

| Name | Ready for Review | Reviewed | Merged | Published |
|-------|-------------------|---------|---------|-----------|
|libp2p |鉂寍鉂寍鉂寍鉂寍
|libp2p-swarm|鉂寍鉂寍鉂寍鉂寍
|ipfs |鉂寍鉂寍鉂寍鉂寍
|libp2p-tcp |鉁厊鉂寍鉂寍鉂寍
|libp2p-websockets |鉁厊鉂寍鉂寍鉂寍
|mafmt |鉁厊鉂寍鉂寍鉂寍

@diasdavid @lgierth @whyrusleeping @dignifiedquire

NOTE: I'm currently bringing this issue up to date with all the latests developments - this is still in WIP.

awesome endeavour statuready
Source
picture dryajov
🎉3 ❤2

All 5 comments

Bumping this as my priority of focus for the next couple of days :) (meaning, will prioritize this over https://github.com/ipfs/js-ipfs/pull/822, so that we are focused in one thing)

daviddias picture daviddias on 16 Apr 2017
🎉1

@diasdavid I've updated the above with some more info. In general circuit should be ready to be reviewed, there might be some minor churn but in general, I don't expect any more major changes. I'll concentrate on finalizing integration with libp2p and js-ipfs, and continue adding more tests.

dryajov picture dryajov on 18 Apr 2017
❤1

@diasdavid

Here are the action items discussed last week:

  • [x] Clean up libp2p tests
  • [x] Add swarm tests
  • [ ] Work on go-js interop tests (WIP)
  • [ ] Prepare demo that showcases circuit (WIP)

    • [ ] Use the files transfer example

dryajov picture dryajov on 30 Aug 2017

@dryajov moved the list to the open PR, reducing the places where information lives https://github.com/ipfs/js-ipfs/pull/830

daviddias picture daviddias on 1 Sep 2017
❤1

Thanks @diasdavid

dryajov picture dryajov on 1 Sep 2017
Was this page helpful?
0 / 5 - 0 ratings

Related issues

valmack picture valmack  路  3Comments
daviddias picture daviddias  路  3Comments
mitra42 picture mitra42  路  3Comments
daviddias picture daviddias  路  3Comments
npfoss picture npfoss  路  3Comments