Skip to content

p2p/discv5: logs info about discv5 node info at bind time#15948

Merged
karalabe merged 1 commit into
ethereum:masterfrom
holiman:addr_v5_bootnode
Jan 23, 2018
Merged

p2p/discv5: logs info about discv5 node info at bind time#15948
karalabe merged 1 commit into
ethereum:masterfrom
holiman:addr_v5_bootnode

Conversation

@holiman
Copy link
Copy Markdown
Contributor

@holiman holiman commented Jan 23, 2018

When -v5 is specified, the node does not output node-info at startup.

The following was show (not -v5) even before this PR:

#build/bin/bootnode -nodekeyhex 1212121212121212121212121212121212121212121212121212121212121212
INFO [01-23|08:51:33] UDP listener up                          self=enode://6360e856310ce5d294e8be33fc807077dc56ac80d95d9cd4ddbd21325eff73f7eb1c2784a65901538479361e94c0a2597973adef0836a6a7eddf50b7997c88a3@[::]:30301

With this PR, -v5 behaves similarly:

#build/bin/bootnode -v5 -nodekeyhex 1212121212121212121212121212121212121212121212121212121212121212
INFO [01-23|08:50:51] UDP listener up                          net=enode://6360e856310ce5d294e8be33fc807077dc56ac80d95d9cd4ddbd21325eff73f7eb1c2784a65901538479361e94c0a2597973adef0836a6a7eddf50b7997c88a3@[::]:30301

@karalabe
Copy link
Copy Markdown
Member

SGTM, but how does this relate to the unified port handling merged yesterday?

@holiman
Copy link
Copy Markdown
Contributor Author

holiman commented Jan 23, 2018

No idea whatsoever. I updated to the most recent code, checked what the problem was, fixed it. Didn't look very unified to me... :trollface:

@karalabe
Copy link
Copy Markdown
Member

;P

@karalabe karalabe merged commit 302c17c into ethereum:master Jan 23, 2018
@karalabe karalabe added this to the 1.8.0 milestone Jan 23, 2018
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.

2 participants