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

LCD is returning no RPC client defined on API calls on develop branch #2907

Closed
2 of 4 tasks
jmdfm opened this issue Nov 27, 2018 · 6 comments · Fixed by #2919
Closed
2 of 4 tasks

LCD is returning no RPC client defined on API calls on develop branch #2907

jmdfm opened this issue Nov 27, 2018 · 6 comments · Fixed by #2919

Comments

@jmdfm
Copy link
Contributor

jmdfm commented Nov 27, 2018

Summary of Bug

It seems that LCD on develop has recently started giving no RPC client defined messages to API calls that work properly on master.

Steps to Reproduce

  1. Check out develop
  2. Build binaries
  3. Launch a local testnet with make localnet-start
  4. Launch LCD via a docker container with this command (make sure to use your chainid):
docker run -v `pwd`:`pwd` -w `pwd` -p 0.0.0.0:1317:1317 --network cosmos-sdk_localnet alpine build/gaiacli --home build/node0/gaiacli rest-server --trust-node=false --laddr tcp://0.0.0.0:1317 --node http://gaiadnode0:26657 --chain-id=chain-5t4HFT
  1. Hit the blocks latest LCD api: curl -k https://0.0.0.0:1317/blocks/latest
  2. See no RPC client defined message.
  3. Checkout master.
  4. Blow away build directory and recompile binaries
  5. Repeat above steps for launching testnet and LCD.
  6. Repeat LCD api query for latest block, see that it responds correctly.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@jackzampolin
Copy link
Member

So I've spent some time looking into this. It looks like where we are setting the cliCtx for the RestServer is not properly pulling in configuration.

@alexanderbez
Copy link
Contributor

alexanderbez commented Nov 27, 2018

@jackzampolin what configuration does it need? The command line arguments? Is this just an order of operations bug?

@jackzampolin
Copy link
Member

@alexanderbez I think so, was trying to debug last night but unsuccessful

@alexanderbez
Copy link
Contributor

Ok, I will look into it.

@alexanderbez
Copy link
Contributor

I cannot reproduce this after my fix in #2918.

@alexanderbez
Copy link
Contributor

In other words, the two are unrelated...sorry to burst the bubble here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants