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

Bianjie/lcd implementation #2118

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
6aa8309
Merge pull request #8 from cosmos/master
Jul 11, 2018
21e36b5
Merge branch 'master' of https://github.com/cosmos/cosmos-sdk
Jul 12, 2018
22161af
Merge branch 'master' of https://github.com/cosmos/cosmos-sdk
Jul 17, 2018
4e61859
Merge branch 'master' of https://github.com/cosmos/cosmos-sdk
Jul 18, 2018
170daf3
Merge branch 'master' of https://github.com/cosmos/cosmos-sdk
Jul 27, 2018
34548db
lcd implemeation
Aug 16, 2018
eccb889
Add golang dependency for swagger
Aug 16, 2018
2047673
Add key rest api
Aug 16, 2018
85f47d2
Add stake query api to lcd swagger
Aug 16, 2018
8e0ca8d
Merge branch 'master' of https://github.com/cosmos/cosmos-sdk
wukongcheng Aug 18, 2018
60acd5c
Merge branch 'bianjie/lcd' of https://github.com/cosmos/cosmos-sdk in…
wukongcheng Aug 18, 2018
187dda4
Merge branch 'bianjie/lcd' of https://github.com/cosmos/cosmos-sdk in…
Aug 18, 2018
22495e9
Merge pull request #58 from HaoyangLiu/haoyang/lcd-implementation
wukongcheng Aug 20, 2018
26af936
Refactor code according to lint result
Aug 21, 2018
4a0d1c5
Merge pull request #63 from HaoyangLiu/haoyang/lcd-implementation
wukongcheng Aug 21, 2018
4607ca9
Refactor comment and fix test failure
Aug 21, 2018
7528534
Add lcd test for swagger lcd
Aug 21, 2018
473fe21
Fix for test lint warnning
Aug 21, 2018
95e367a
Refactor comment
Aug 21, 2018
9d944b0
Add new test for lcd with swagger
Aug 22, 2018
fc40419
Merge pull request #66 from HaoyangLiu/haoyang/lcd-implementation
wukongcheng Aug 22, 2018
f266472
Refactor lcd according to code review
Aug 23, 2018
6cdf21d
Refactor code according to code review
Aug 24, 2018
79ec713
Merge pull request #73 from HaoyangLiu/haoyang/lcd-implementation-new
wukongcheng Aug 24, 2018
9409e73
Fix errors in test_cover and test_lint
Aug 24, 2018
9ff57af
Merge pull request #77 from HaoyangLiu/haoyang/lcd-implementation-new
wukongcheng Aug 24, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions Gopkg.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions Gopkg.toml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,14 @@
name = "github.com/stretchr/testify"
version = "=1.2.1"

[[constraint]]
name = "github.com/swaggo/gin-swagger"
version = "=v1.0.0"

[[constraint]]
name = "github.com/swaggo/swag"
version = "=v1.3.2"

[[override]]
name = "github.com/tendermint/go-amino"
version = "=0.10.1"
Expand Down
15 changes: 15 additions & 0 deletions client/context/context.go
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import (
"github.com/spf13/viper"

rpcclient "github.com/tendermint/tendermint/rpc/client"
tendermintLite"github.com/tendermint/tendermint/lite"
)

const ctxAccStoreName = "acc"
Expand All @@ -30,6 +31,8 @@ type CLIContext struct {
Async bool
JSON bool
PrintResponse bool
Certifier tendermintLite.Certifier
ClientManager *ClientManager
}

// NewCLIContext returns a new initialized CLIContext with parameters from the
Expand Down Expand Up @@ -113,3 +116,15 @@ func (ctx CLIContext) WithUseLedger(useLedger bool) CLIContext {
ctx.UseLedger = useLedger
return ctx
}

// WithCertifier - return a copy of the context with an updated Certifier
func (ctx CLIContext) WithCertifier(certifier tendermintLite.Certifier) CLIContext {
ctx.Certifier = certifier
return ctx
}

// WithClientManager - return a copy of the context with an updated ClientManager
func (ctx CLIContext) WithClientManager(clientManager *ClientManager) CLIContext {
ctx.ClientManager = clientManager
return ctx
}
46 changes: 46 additions & 0 deletions client/context/loadbalancing.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
package context

import (
rpcclient "github.com/tendermint/tendermint/rpc/client"
"strings"
"sync"
"github.com/pkg/errors"
)

// ClientManager is a manager of a set of rpc clients to full nodes.
// This manager can do load balancing upon these rpc clients.
type ClientManager struct {
clients []rpcclient.Client
currentIndex int
mutex sync.RWMutex
}

// NewClientManager create a new ClientManager
func NewClientManager(nodeURIs string) (*ClientManager,error) {
if nodeURIs != "" {
nodeURLArray := strings.Split(nodeURIs, ",")
var clients []rpcclient.Client
for _, url := range nodeURLArray {
client := rpcclient.NewHTTP(url, "/websocket")
clients = append(clients, client)
}
mgr := &ClientManager{
currentIndex: 0,
clients: clients,
}
return mgr, nil
}
return nil, errors.New("missing node URIs")
}

func (mgr *ClientManager) getClient() rpcclient.Client {
Copy link
Collaborator

Choose a reason for hiding this comment

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

(mgr *ClientManager) ––> (manager *ClientManager)

mgr.mutex.Lock()
defer mgr.mutex.Unlock()

client := mgr.clients[mgr.currentIndex]
mgr.currentIndex++
if mgr.currentIndex >= len(mgr.clients){
mgr.currentIndex = 0
}
return client
}
16 changes: 16 additions & 0 deletions client/context/loadbalancing_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
package context

import (
"testing"
"github.com/stretchr/testify/assert"
)

func TestLoadBalancing(t *testing.T) {
nodeURIs := "10.10.10.10:26657,20.20.20.20:26657,30.30.30.30:26657"
clientMgr,err := NewClientManager(nodeURIs)
assert.Empty(t,err)
endpoint := clientMgr.getClient()
assert.NotEqual(t,endpoint,clientMgr.getClient())
clientMgr.getClient()
assert.Equal(t,endpoint,clientMgr.getClient())
}
77 changes: 76 additions & 1 deletion client/context/query.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,19 @@ import (
cmn "github.com/tendermint/tendermint/libs/common"
rpcclient "github.com/tendermint/tendermint/rpc/client"
ctypes "github.com/tendermint/tendermint/rpc/core/types"
"github.com/cosmos/cosmos-sdk/store"
"github.com/cosmos/cosmos-sdk/wire"
"strings"
tendermintLiteProxy "github.com/tendermint/tendermint/lite/proxy"
abci "github.com/tendermint/tendermint/abci/types"
)

// GetNode returns an RPC client. If the context's client is not defined, an
// error is returned.
func (ctx CLIContext) GetNode() (rpcclient.Client, error) {
if ctx.ClientManager != nil {
return ctx.ClientManager.getClient(), nil
}
if ctx.Client == nil {
return nil, errors.New("no RPC client defined")
}
Expand Down Expand Up @@ -277,6 +285,50 @@ func (ctx CLIContext) ensureBroadcastTx(txBytes []byte) error {
return nil
}

// proofVerify perform response proof verification
func (ctx CLIContext) proofVerify(path string, resp abci.ResponseQuery) error {
// Data from trusted node or subspace query doesn't need verification
if ctx.TrustNode || !isQueryStoreWithProof(path) {
return nil
}

// TODO: Later we consider to return error for missing valid certifier to verify data from untrusted node
if ctx.Certifier == nil {
if ctx.Logger != nil {
io.WriteString(ctx.Logger, fmt.Sprintf("Missing valid certifier to verify data from untrusted node\n"))
}
return nil
}

node, err := ctx.GetNode()
if err != nil {
return err
}
// AppHash for height H is in header H+1
commit, err := tendermintLiteProxy.GetCertifiedCommit(resp.Height+1, node, ctx.Certifier)
if err != nil {
return err
}

var multiStoreProof store.MultiStoreProof
cdc := wire.NewCodec()
err = cdc.UnmarshalBinary(resp.Proof, &multiStoreProof)
if err != nil {
return errors.Wrap(err, "failed to unmarshalBinary rangeProof")
}

// Validate the substore commit hash against trusted appHash
substoreCommitHash, err := store.VerifyMultiStoreCommitInfo(multiStoreProof.StoreName, multiStoreProof.CommitIDList, commit.Header.AppHash)
if err != nil {
return errors.Wrap(err, "failed in verifying the proof against appHash")
}
err = store.VerifyRangeProof(resp.Key, resp.Value, substoreCommitHash, &multiStoreProof.RangeProof)
if err != nil {
return errors.Wrap(err, "failed in the range proof verification")
}
return nil
}

// query performs a query from a Tendermint node with the provided store name
// and path.
func (ctx CLIContext) query(path string, key common.HexBytes) (res []byte, err error) {
Expand All @@ -296,10 +348,15 @@ func (ctx CLIContext) query(path string, key common.HexBytes) (res []byte, err e
}

resp := result.Response
if !resp.IsOK() {
if resp.Code != uint32(0) {
return res, errors.Errorf("query failed: (%d) %s", resp.Code, resp.Log)
}

err = ctx.proofVerify(path, resp)
if err != nil {
return nil, err
}

return resp.Value, nil
}

Expand All @@ -309,3 +366,21 @@ func (ctx CLIContext) queryStore(key cmn.HexBytes, storeName, endPath string) ([
path := fmt.Sprintf("/store/%s/%s", storeName, endPath)
return ctx.query(path, key)
}

// isQueryStoreWithProof expects a format like /<queryType>/<storeName>/<subpath>
// queryType can be app or store
// if subpath equals to store or key, then return true
func isQueryStoreWithProof(path string) (bool) {
if !strings.HasPrefix(path, "/") {
return false
}
paths := strings.SplitN(path[1:], "/", 3)
if len(paths) != 3 {
return false
}
// WARNING This should be consistent with query method in iavlstore.go
if paths[2] == "store" || paths[2] == "key" {
return true
}
return false
}
4 changes: 4 additions & 0 deletions client/flags.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@ const (
FlagAsync = "async"
FlagJson = "json"
FlagPrintResponse = "print-response"
FlagListenAddr = "laddr"
FlagSwaggerHostIP = "swagger-host-ip"
FlagModules = "modules"
FlagNodeList = "node-list"
)

// LineBreak can be included in a command list to provide a blank line
Expand Down
32 changes: 32 additions & 0 deletions client/httputils/httputils.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
package httputils

import (
"github.com/gin-gonic/gin"
"net/http"
)

// NewError create error http response
func NewError(ctx *gin.Context, errCode int, err error) {
errorResponse := HTTPError{
API: "2.0",
Code: errCode,
}
if err != nil {
errorResponse.ErrMsg = err.Error()
}

ctx.JSON(errCode, errorResponse)
}

// NormalResponse create normal http response
func NormalResponse(ctx *gin.Context, data []byte) {
ctx.Status(http.StatusOK)
ctx.Writer.Write(data)
}

// HTTPError is http response with error
type HTTPError struct {
API string `json:"rest api" example:"2.0"`
Code int `json:"code" example:"500"`
ErrMsg string `json:"error message"`
}
Loading