Skip to content
This repository has been archived by the owner on Aug 23, 2019. It is now read-only.

Async Crypto Endeavour #108

Merged
merged 8 commits into from
Nov 3, 2016
Merged

Async Crypto Endeavour #108

merged 8 commits into from
Nov 3, 2016

Conversation

dignifiedquire
Copy link
Member

(err, id) => {
if (err) {
return cb(err)
}
Copy link
Member

Choose a reason for hiding this comment

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

It would be super beneficial to have a libp2p module libp2p-peer-gen that already has 100 or 1000 PeerId generated and so, each time we request 'one peer', it would be super fast because it only had to load it from disk. We would use this everywhere for testing!

Copy link
Member

Choose a reason for hiding this comment

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

🛎

let peerB

before((done) => {
utils.createInfos(2, (err, infos) => {
Copy link
Member

Choose a reason for hiding this comment

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

It seems you already went ahead and did what I proposed above. Would you like to make it a module?

cb()
})
})
}
Copy link
Member

Choose a reason for hiding this comment

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

Let's make this read from disk instead :)

Copy link
Member Author

Choose a reason for hiding this comment

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

okay, will pull out and make its own module with pre generated 100 ids, and if it needs more generate them on the fly

Copy link
Member Author

Choose a reason for hiding this comment

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

I've left it in here for now, but it has now ids on disk that it tries to read.

Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

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

Only missing the Sauce Labs testing badge

@daviddias daviddias changed the title refactor: use async peer-info and peer-id Async Crypto Endeavour Oct 30, 2016
@dignifiedquire
Copy link
Member Author

Only missing the Sauce Labs testing badge

done

@daviddias daviddias merged commit 0c1d887 into master Nov 3, 2016
@daviddias daviddias deleted the fast-keys branch November 3, 2016 09:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants