Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Allow deterministic key generation from seed #130

Merged
merged 3 commits into from
Aug 2, 2021
Merged

Conversation

thattommyhall
Copy link
Member

I think this addresses #128 quite neatly, I took some advice from @Kubuxu to use the HKDF and looking over how we did stuff in go-libp2p-core/crypto but im pretty new to Go

@thattommyhall thattommyhall force-pushed the deterministic-peer-ids branch from e39fc3f to f79ca70 Compare July 24, 2021 00:49
Copy link
Contributor

@petar petar left a comment

Choose a reason for hiding this comment

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

Very nice solution.

Copy link
Member

@Kubuxu Kubuxu left a comment

Choose a reason for hiding this comment

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

This will work nicely if the only thing using that reader is the key generation and it is read in somewhat of deterministic order and it looks like it is.

@thattommyhall thattommyhall requested a review from petar July 30, 2021 22:57
Copy link
Contributor

@petar petar left a comment

Choose a reason for hiding this comment

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

lgtm

@petar petar merged commit 385df0e into master Aug 2, 2021
@petar petar deleted the deterministic-peer-ids branch August 2, 2021 13:05
@petar petar mentioned this pull request Aug 2, 2021
9 tasks
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.

3 participants