-
Notifications
You must be signed in to change notification settings - Fork 87
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
Add poke to state network spec #347
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR explains well how/when POKE can be executed in the Portal state network.
But it does not state that it MUST, SHOULD or MAY be executed (and/or in which occasion). This keeps it quite ambiguous on whether a client should/must implement this or not.
So I think as a bare minimum something should be added like: "POKE mechanism SHOULD be executed when the proof is available on the node/application.
(Intuitively I'm also a little bit on the fence regarding how "application space" gets directly involved here in maintaining the health of the network. But this hesitation is probably too unfounded and it should help for the health of the network.) edit: It is not really true what I wrote there so ignore that, application side does not know about this happening, at least not for higher level calls like getAccount and so on.
Good point. In my opinion, clients SHOULD implement poke because it should be encouraged in order to spread data across the network. I've updated with additional text as suggested. |
@kdeme @KolbyML @morph-dev Did you have any thoughts on this one? I have already implemented poke in the state network in Fluffy (ready in a PR) so I'm keen to have this be part of the protocol for state network. I'm not aware of any reason not to implement it. |
Co-authored-by: Kim De Mey <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Poke in the state network is feasible by passing in the parent offer when looking up content. The fetched state content is then combined with the parent offer to build the offer which is used in the poke.
I'm not aware of any reason why poke should still be disabled in the state network, so I suggest we re-enable it.