Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Net etiquette #1028

Merged
merged 1 commit into from
May 2, 2016
Merged

Net etiquette #1028

merged 1 commit into from
May 2, 2016

Conversation

arkpar
Copy link
Collaborator

@arkpar arkpar commented May 2, 2016

  • Track useless peers and don't attempt connecting to them more often than once in 5 minutes.
  • Send out disconnect packet properly if the connection is closed on out side.

@arkpar arkpar added the A0-pleasereview 🤓 Pull request needs code review. label May 2, 2016
@@ -204,6 +205,7 @@ impl NodeTable {
NodeTable {
path: path.clone(),
nodes: NodeTable::load(path),
uselsess_nodes: HashSet::new(),
Copy link
Contributor

Choose a reason for hiding this comment

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

spelling

@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels May 2, 2016
@gavofyork gavofyork merged commit c34e353 into master May 2, 2016
@gavofyork gavofyork deleted the net branch May 2, 2016 12:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants