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

Bug when specifying protocol in host.* in config.yml #1102

Closed
fadeev opened this issue May 6, 2021 · 2 comments
Closed

Bug when specifying protocol in host.* in config.yml #1102

fadeev opened this issue May 6, 2021 · 2 comments
Assignees

Comments

@fadeev
Copy link
Contributor

fadeev commented May 6, 2021

host:
  api: "http://192.168.1.208:1317"
  rpc: "http://192.168.1.208:26657"

Results in tcp://http://192.168.1.208:1317 in config.toml.

We may need to revisit the host property. Currently, we have:

RPC:      "0.0.0.0:26657",
P2P:      "0.0.0.0:26656",
Prof:     "0.0.0.0:6060",
GRPC:     "0.0.0.0:9090",
API:      "0.0.0.0:1317",
Frontend: "0.0.0.0:8080",
DevUI:    "0.0.0.0:12345",

RPC, P2P, prof, gRPC, and API can be configure through init.config and init.app. Frontend and Dev UI are no longer relevant. I'm wondering whether we need the host property. We can keep it, of course, but there is still confusion when users try to set init.app.api.address and it doesn't get set in app.toml, because we overwrite it with our default.

@andynog
Copy link

andynog commented May 6, 2021

Removing the http:// seems to work

host:
  api: "192.168.1.208:1317"
  rpc: "192.168.1.208:26657"

@jeronimoalbi
Copy link
Member

jeronimoalbi commented Feb 7, 2024

Config format changed since this issue was created, Cosmos app and Tendermint settings are configured using the validators config options. Each validator has the app and config map properties.
Just tested the config using the described values and verified that the config.toml and app.toml files are generated correctly.

@salmad3 I think we can close this issue.

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

No branches or pull requests

4 participants