Skip to content
This repository was archived by the owner on Aug 29, 2023. It is now read-only.

WIP - Feat/circuit interop #81

Merged
merged 3 commits into from
Oct 13, 2017
Merged

WIP - Feat/circuit interop #81

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c1fdaa3
feat: filter IPFS addrs correctly
dryajov Apr 2, 2017
5652591
feat: mafmt addrs now support /ipfs no need for ad-hoc filtering
dryajov Aug 29, 2017
23a7cbe
feat: skip p2p-circuit addresses
dryajov Sep 14, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
feat: mafmt addrs now support /ipfs no need for ad-hoc filtering
  • Loading branch information
@dryajov
dryajov committed Sep 3, 2017
commit 56525912f649477e73cf87af9fa196ec17fdc007
2 changes: 1 addition & 1 deletion src/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ class TCP {
multiaddrs = [multiaddrs]
}
return multiaddrs.filter((ma) => {
return mafmt.TCP.matches(ma) || mafmt.TCP_IPFS.matches(ma)
return mafmt.TCP.matches(ma)
Copy link
Member

@daviddias daviddias Sep 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Notes from the call today:

It should continue to be valid to pass an ipfs multiaddr, it just needs to check things differently.

    1. check if it is relay, if it is, skip it
    1. if not relay, check if it is ipfs, if it is, decapsulate
    1. check if it is TCP, if it is, include it

Copy link
Member Author

@dryajov dryajov Sep 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@diasdavid ii and iii already happen in master, so it should be adding a check for circuit and skipping it if its present (i).

})
}
}
Expand Down