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

Add a websocket session RPC. #500

Merged
merged 2 commits into from
Sep 17, 2015
Merged

Conversation

jrick
Copy link
Member

@jrick jrick commented Sep 16, 2015

This will be used in an upcoming btcwallet change to detect disconnected (or reconnected) websocket connections to the btcd RPC server.

@davecgh
Copy link
Member

davecgh commented Sep 16, 2015

The code looks good, but would you also update the Websocket Extension Methods section of the JSON RPC API docs?

@jrick jrick force-pushed the jrick_session branch 3 times, most recently from 86856bf to bb98bbb Compare September 17, 2015 15:24

// SessionResult models the data from the session command.
type SessionResult struct {
SessionID uint64 `json:sessionid`
Copy link
Member

Choose a reason for hiding this comment

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

json:"sessionid"

@davecgh
Copy link
Member

davecgh commented Sep 17, 2015

OK

@conformal-deploy conformal-deploy merged commit 3c9d18d into btcsuite:master Sep 17, 2015
jcvernaleo pushed a commit to jcvernaleo/btcd that referenced this pull request Dec 6, 2016
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.

3 participants