Skip to content
This repository has been archived by the owner on Jun 25, 2021. It is now read-only.

Remove state on ::connect #711

Closed
benjaminbollen opened this issue Sep 21, 2015 · 1 comment · Fixed by #712
Closed

Remove state on ::connect #711

benjaminbollen opened this issue Sep 21, 2015 · 1 comment · Fixed by #712
Assignees
@benjaminbollen
Labels
bug critical

Comments

@benjaminbollen
Copy link

with the transition to the new Crust 0.3 API, state has been introduced with RoutingNode::connect_requests that can not be there. It should be ok to remove this state by relying on the directionality of the ::crust::Event::OnAccept and ::crust::Event::OnConnect

https://github.com/maidsafe/routing/pull/704/files#diff-1ee67c5acfa53a1ba6d9a5bb56540ba2R968
@benjaminbollen benjaminbollen self-assigned this Sep 21, 2015
@benjaminbollen
Copy link
Author

#704 also needs an additional correction on routing bootstrapping as it currently does not work.

@benjaminbollen benjaminbollen mentioned this issue Sep 22, 2015
Merged
@ghost ghost closed this as completed in #712 Sep 22, 2015
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@benjaminbollen benjaminbollen

Labels
bug critical
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue 711 benjaminbollen/routing
2 participants
@dirvine @benjaminbollen