Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Configuration To Include Noise Protocol And Private Networks #51

Merged
merged 5 commits into from
Mar 17, 2020

Conversation

bonedaddy
Copy link
Collaborator

@bonedaddy bonedaddy commented Mar 13, 2020

Document usage of the noise protocol, and indicate how private networks can be triggered

@bonedaddy bonedaddy changed the title Update Configuration To Include Noise Protocol Update Configuration To Include Noise Protocol And Private Networks Mar 13, 2020
@@ -150,6 +150,9 @@ node:
# enables the quic transport
# default is false
quic: false
# enables the noisee security transport
Copy link
Contributor

Choose a reason for hiding this comment

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

double e

Comment on lines 168 to 171
# enables private libp2p swarm mode
# the value here will be used as the private network shared secret key
# default is empty string ("") which means use the public network
swarm_key: 6f0cf24809cda96d0d68b0fa8abdd64eccd1b4866262040cdc5afecba3418aad
Copy link
Contributor

Choose a reason for hiding this comment

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

Should specify the format and size of the key here (32byte, hex). Also, the sample key should be a fake key or not given, otherwise it will get used unintentionally.

@codecov
Copy link

codecov bot commented Mar 17, 2020

Codecov Report

Merging #51 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #51   +/-   ##
=======================================
  Coverage   81.56%   81.56%           
=======================================
  Files           7        7           
  Lines         347      347           
=======================================
  Hits          283      283           
  Misses         37       37           
  Partials       27       27

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 266caa1...e493bcd. Read the comment docs.

@bonedaddy bonedaddy merged commit e83d2ee into master Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants