Skip to content

Conversation

@najuna-brian
Copy link
Contributor

Closes #1945

Description

Clarifies when to use the Docker Helper URL (…local-ip.medicmobile.org:PORT, valid TLS) vs https://localhost (self-signed) in local setup and cht-conf examples. Adds a tip and tabbed examples to prevent URL/cert errors.

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

@najuna-brian najuna-brian marked this pull request as ready for review September 21, 2025 13:56
@mrjones-plip mrjones-plip self-requested a review September 22, 2025 19:19
Copy link
Contributor

@mrjones-plip mrjones-plip left a comment

Choose a reason for hiding this comment

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

Nice work here - thank you!

Requesting to changes to get this tidied up a bit.

Copy link
Contributor

@mrjones-plip mrjones-plip left a comment

Choose a reason for hiding this comment

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

Thanks @najuna-brian ! I think maybe reducing the URL to have the localhost IP and just requesting they update the port might be easier. I've made some suggestions to this end.

Comment on lines +216 to +219

{{< callout type="tip" >}}
Docker Helper users: copy the exact `...local-ip.medicmobile.org:PORT` URL that the helper prints and use it in all `cht` commands. With this URL, you do not need `--accept-self-signed-certs` because the certificate is valid.
{{< /callout >}}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this call out is a great idea! However, I think it will be more contextual when it's directly next two the your-local-ip type URLs. Let's remove it here and add an inline call out below:

Suggested change
{{< callout type="tip" >}}
Docker Helper users: copy the exact `...local-ip.medicmobile.org:PORT` URL that the helper prints and use it in all `cht` commands. With this URL, you do not need `--accept-self-signed-certs` because the certificate is valid.
{{< /callout >}}

Comment on lines +234 to +237

```shell
cht --url=https://medic:password@<your-local-ip.medicmobile.org:PORT> csv-to-docs upload-docs
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
```shell
cht --url=https://medic:password@<your-local-ip.medicmobile.org:PORT> csv-to-docs upload-docs
```
Be sure to update the `<PORT>` to be correct. It will be a 5 digits number like `10443`. The full URL is show right after you [start a docker helper instance]({{< relref "/hosting/cht/app-developer#cht-docker-helper" >}}).
```shell
cht --url=https://medic:password@127-0-0-1.local-ip.medicmobile.org:<PORT> csv-to-docs upload-docs
```

Comment on lines 264 to 266
```shell
# accept-self-signed-certs bypasses normal SSL certificate verification. This is necessary when connecting to a local CHT instance.
cht --url=https://medic:password@localhost --accept-self-signed-certs
cht --url=https://medic:password@<your-local-ip.medicmobile.org:PORT> compile-app-settings upload-app-settings
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
```shell
# accept-self-signed-certs bypasses normal SSL certificate verification. This is necessary when connecting to a local CHT instance.
cht --url=https://medic:password@localhost --accept-self-signed-certs
cht --url=https://medic:password@<your-local-ip.medicmobile.org:PORT> compile-app-settings upload-app-settings
```
Be sure to update the `<PORT>` to be correct. It will be a 5 digits number like `10443`. The full URL is show right after you [start a docker helper instance]({{< relref "/hosting/cht/app-developer#cht-docker-helper" >}}).
```shell
cht --url=https://medic:password@127-0-0-1.local-ip.medicmobile.org:<PORT> csvcompile-app-settings upload-app-settings
```

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify local setup instructions for Docker users to avoid URL confusion

2 participants