Merge pull request #60 from evanlinjin/bug/hypervisor-#29

Added 'online' field to hypervisor get node(s) response.
skycoin · Nov 11, 2019 · fee6453 · fee6453
2 parents 21251cb + 283ae09
commit fee6453
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/pkg/hypervisor/hypervisor.go b/pkg/hypervisor/hypervisor.go
@@ -231,6 +231,7 @@ func (m *Node) exec() http.HandlerFunc {
 
 type summaryResp struct {
 	TCPAddr string `json:"tcp_addr"`
+	Online  bool   `json:"online"`
 	*visor.Summary
 }
 
@@ -247,6 +248,7 @@ func (m *Node) getNodes() http.HandlerFunc {
 			}
 			summaries = append(summaries, summaryResp{
 				TCPAddr: c.Addr.Addr.String(),
+				Online:  err == nil,
 				Summary: summary,
 			})
 		}

diff --git a/pkg/visor/rpc.go b/pkg/visor/rpc.go
@@ -137,7 +137,7 @@ func newTransportSummary(tm *transport.Manager, tp *transport.ManagedTransport,
 	return summary
 }
 
-// Summary provides a summary of an AppNode.
+// Summary provides a summary of a Skywire Visor.
 type Summary struct {
 	PubKey          cipher.PubKey       `json:"local_pk"`
 	NodeVersion     string              `json:"node_version"`