-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
p2p: report-peer-id option for listen #5771
Conversation
59721f7
to
fa2e6c5
Compare
We should also consider sending a binary peer ID (maybe length-prefixed?, although peer IDs are already prefix-free codes). This may be easier to parse than having to read to a newline and then potentially base58 decode. It also means we can change encoding without breaking things. However, I haven't thought though this thoroughly. |
Technically they carry the multihash prefix which has length. I can see binary form being useful, but I can't think of any nice way to do this other than 2 separate flags |
Meh. The current version is quite user friendly. Any objections @ipfs/go-team? |
Eh, I think that practically, the base58 encoded version will be the most useful. If we make it binary, writing a parser becomes just that much more difficult. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A week has gone by, @whyrusleeping has given his blessing, 🚂.
fa2e6c5
to
39515af
Compare
License: MIT Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
License: MIT Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
39515af
to
d6626a5
Compare
Was suggested on IRC, when set it basically passes
QmPeerid\n
of connecting peer into listener stream.(works but needs tests)