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

OnVersion now supports returning a Reject Message to peer (which will… #111

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jadeblaquiere
Copy link

… be disconnected after Reject message is sent)

see : btcsuite/btcd@7b103e2

Instead of simply disconnecting, compose a disconnect message so the peer is given a reason for the disconnect.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 70.052% when pulling e11a473 on jadeblaquiere:reject_msg into 641af1c on lightninglabs:master.

@halseth
Copy link
Contributor

halseth commented Dec 5, 2018

Needs a rebase!

@jadeblaquiere
Copy link
Author

jadeblaquiere commented Dec 6, 2018

Not sure what's going on with Travis builds... " fatal: unable to access 'https://go.googlesource.com/crypto/': The requested URL returned error: 502"

This seems to be an environmental failure... perhaps simply ask Travis to try harder?

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.

3 participants