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

tag 'can-hop' peers to aid in connection closing logic #20

Merged
merged 1 commit into from Oct 10, 2017

Conversation

whyrusleeping
Copy link
Contributor

@whyrusleeping whyrusleeping commented Oct 8, 2017

This should also probably tag relay peers with a higher value when they are actually relaying a connection for us. The logic there isnt simple, so i've punted on it for now.

@vyzo
Copy link
Contributor

vyzo commented Oct 10, 2017

this will tag all connections to relays effectively, is that what we want?

@whyrusleeping
Copy link
Contributor Author

whyrusleeping commented Oct 10, 2017

@vyzo
Copy link
Contributor

vyzo commented Oct 10, 2017

we should certainly tag connections that are actively relayed with high priority.
what does the value of 2 mean priority-wise?

@whyrusleeping
Copy link
Contributor Author

whyrusleeping commented Oct 10, 2017

vyzo
vyzo approved these changes Oct 10, 2017
@vyzo vyzo merged commit 6641cf5 into master Oct 10, 2017
2 checks passed
@vyzo vyzo deleted the feat/conn-tagging branch Oct 10, 2017
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