Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(crates/tuono_lib): print server url as valid URL to be clickable on CLI #464

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

marcalexiei
Copy link
Member

Checklist

Related issue

Fixes #460

Overview

  • restore protocol at the beginning of server ready message
  • renamed server_http_address removing the http since it's actually a hostname + port and doesn't have a protocol
    (this is out of scope, but IMHO it's a nice to have, let me know if I need to revert it)

@marcalexiei marcalexiei self-assigned this Jan 29, 2025
@github-actions github-actions bot added the rust Requires rust knowledge label Jan 29, 2025
@marcalexiei marcalexiei force-pushed the format-server—base-url branch from 02ec229 to 829c3b7 Compare January 29, 2025 12:43
@marcalexiei marcalexiei marked this pull request as ready for review January 29, 2025 12:46
Copy link
Member

@Valerioageno Valerioageno left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

@Valerioageno Valerioageno merged commit eb9039d into main Jan 29, 2025
17 checks passed
@Valerioageno Valerioageno deleted the format-server—base-url branch January 29, 2025 16:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
rust Requires rust knowledge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: 0.17.6 - tuono server url is no longer contains the protocol
2 participants