Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Conversation

@arkpar
Copy link
Member

@arkpar arkpar commented Apr 10, 2019

This restores peer banning for e.g. being on a different chain, until this is properly implemented.

@arkpar arkpar added A0-please_review Pull request needs code review. B0-patchthis labels Apr 10, 2019
@Demi-Marie
Copy link
Contributor

I am about to test this.

@bkchr bkchr merged commit 9ab407d into master Apr 10, 2019
@bkchr bkchr deleted the a-ban-for-good branch April 10, 2019 21:34
MTDK1 pushed a commit to bdevux/substrate that referenced this pull request Apr 12, 2019
* Apply negative rating on explicit ban

* Update core/network/src/service.rs

Co-Authored-By: arkpar <[email protected]>
cmichi pushed a commit that referenced this pull request Apr 13, 2019
* Apply negative rating on explicit ban

* Update core/network/src/service.rs

Co-Authored-By: arkpar <[email protected]>
andresilva pushed a commit that referenced this pull request Apr 15, 2019
* Apply negative rating on explicit ban

* Update core/network/src/service.rs

Co-Authored-By: arkpar <[email protected]>
andresilva pushed a commit that referenced this pull request Apr 15, 2019
* Apply negative rating on explicit ban

* Update core/network/src/service.rs

Co-Authored-By: arkpar <[email protected]>
gavofyork pushed a commit that referenced this pull request Apr 16, 2019
* simplification of peerset api (#2123)

* Introduction of PeersetHandle

* integrate PeersetHandle with the rest of the codebase

* fix compilation errors

* more tests for peerset, fixed overwriting bug in add_reserved_peer

* Slots data structure and bugfixes for peerset

* bend to pressure

* updated lru-cache to 0.1.2 and updated linked-hash-map to 0.5.2

* peerset discovered list is now a LinkedHashMap

* fix review suggestions

* split back Peerset and PeersetHandle

* test for Peerset::discovered

* applied review suggestions

* fixes to peerset::incoming

* peerset disconnects are all instantaneous

* instantaneous drop in peerset finished

* Peerset::set_reserved_only can also reconnect nodes

* Peerset scores cache uses lru-cache

* remove redundant function call and comment from Peerset::on_set_reserved_only

* add_peer returns SlotState enum

* apply review suggestions

* is_reserved -> is_connected_and_reserved

* Fix state inconsistency between handler and behaviour (#2220)

* Fix state inconsistency between handler and behaviour

* Fix the error! being in the wrong place

* Apply negative rating on explicit ban (#2247)

* Apply negative rating on explicit ban

* Update core/network/src/service.rs

Co-Authored-By: arkpar <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants