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

Improve boot related cli output #938

Merged
merged 8 commits into from
Dec 4, 2024
Merged

Conversation

James-Mart
Copy link
Member

@James-Mart James-Mart commented Dec 2, 2024

psinode start message

Old message on initial execution of psinode: Need genesis block; use 'psibase boot' to boot chain
New message:
Node is not connected to any psibase network.

If the x-admin app is hosted at a subdomain, the message is further improved to:
Node is not connected to any psibase network. Visit '$protocol//$subdomain.psibase.127.0.0.1.sslip.io:$port' for node setup

  • Where $protocol is https: if there is an https listen directive, http: otherwise.
  • Where $subdomain is the subdomain at which the x-admin app is hosted, (usually x-admin.)
  • Where $port is the listen port, https preferred, http fallback

For example, message might be:
Node is not connected to any psibase network. Visit 'https://x-admin.psibase.127.0.0.1.sslip.io:8070' for node setup

psibase boot message

Old message: Ok.
New message: Successfully booted $api

  • Where $api is whatever URL was passed to the -a flag during boot

For example:
Successfully booted http://psibase.127.0.0.1.sslip.io:8069/

@James-Mart James-Mart requested a review from swatanabe December 2, 2024 18:18
@James-Mart James-Mart marked this pull request as ready for review December 2, 2024 18:18
programs/psinode/main.cpp Outdated Show resolved Hide resolved
rust/psibase/src/main.rs Outdated Show resolved Hide resolved
programs/psinode/main.cpp Outdated Show resolved Hide resolved
programs/psinode/main.cpp Outdated Show resolved Hide resolved
@James-Mart James-Mart added the Dev Experience Related to the experience of developers label Dec 4, 2024
@James-Mart James-Mart merged commit 60d3617 into main Dec 4, 2024
3 checks passed
@James-Mart James-Mart deleted the improve-boot-related-CLI-output branch December 4, 2024 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dev Experience Related to the experience of developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants