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 documentation to specify url of DefraDB node on remote installs #788

Closed
fredcarle opened this issue Sep 12, 2022 · 2 comments · Fixed by #814
Closed

Add documentation to specify url of DefraDB node on remote installs #788

fredcarle opened this issue Sep 12, 2022 · 2 comments · Fixed by #814
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@fredcarle
Copy link
Collaborator

DJ and Cameron | TCB where jamming together to figure out an issue Cameron was having with a remote install on an AWS EC2 instance. It turns out that the VM might not publish the open port when using localhost (still need to investigate this) so the private IP of the VM should be specified for the DefraDB node.

@fredcarle fredcarle added the documentation Improvements or additions to documentation label Sep 12, 2022
@fredcarle fredcarle added this to the DefraDB v0.3.1 milestone Sep 12, 2022
@orpheuslummis
Copy link
Contributor

The param --url string URL of the target database's HTTP endpoint (default "localhost:9181") has two semantics, to determine the address at which the HTTP API is provided, and or determine the address of a remote DB to connect to. In the CLI it is a param on the root command.

It is possible to:

  • break it down into two different params
  • in the CLI, have them on subcommands instead of root
  • document it/them better

@orpheuslummis
Copy link
Contributor

For posterity & completeness:

  • localhost, 127.0.0.1, ::1 are aliases for the loopback interface, i.e. machine talking with itself only
  • 0.0.0.0, :: are aliases to 'listen on an external network interfaces'

For security reasons, the default config should be to listen on loopback.

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

Successfully merging a pull request may close this issue.

2 participants