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

Update documentation for cases when server has no public DNS name #186

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

eguchek
Copy link

@eguchek eguchek commented Dec 3, 2021

Just a suggestion what to do with the error during selfhost-clearnet-certbot configuration if server has no public DNS name

Copy link
Collaborator

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

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

Thanks for PR! I think it's a good idea.

@@ -197,7 +197,7 @@ BTC RPC Explorer is a very powerful Bitcoin explorer that uses your own full nod
* Tor onion domain is generated by default.
* Install `selfhost-clearnet` to activate clearnet domain **continue reading the docs!**
* Installation of `selfhost-clearnet` **requires interaction** because it asks for domain.
* **You must have a working domain pointing to your node for `selfhost-clearnet` to work!**
* **You must have a working domain pointing to your node for `selfhost-clearnet` to work!** Otherwise you can remove `selfhost-clearnet-certbot` to prevent certificate retrieval error during package configuration
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe instead of "Otherwise" write "If you installed it by mistake", I think it should be clearer. What do you think?

Copy link
Author

Choose a reason for hiding this comment

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

In my case selfhost-clearnet-certbot wasn't installed by mistake. It just comes together with selfhost-clearnet. Just checked it in docker:
image

Copy link
Collaborator

Choose a reason for hiding this comment

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

Well, selfhost-clearnet doesn't work with working domain either (unless you path /etc/hosts but that's an edge case), so even installing that one is probably a mistake.

Copy link
Author

Choose a reason for hiding this comment

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

Probably it is an undocumented hack, but selfhost-clearnet works fine if you provide ipv4 address as the domain name. The only issue is that selfhost-clearnet-certbot fails to configure in such case. So, removing selfhost-clearnet-certbot makes dpkg happy and keeps selfhost-clearnet available by ip address via http. It's definetly not a secure way to use selfhost dashboard, but the esiest to get it up and running. Maybe it makes sense to describe this way in documentation as possible but not recommended?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh, it didn't occur to me it could be possible. I agree with documenting it is possible but a bad idea due to security issues. I also wouldn't be surprised if some applications didn't work.

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.

2 participants