Shell needs to understand and show libsql URLs

CodingDoug commented 1 year ago

The shell does not understand libsql URLs, so copy and pasting a URL isn't effective at all.

The shell also shows HTTP URLs with credentials when started. Instead it should probably show a libsql URL, or the actual URL given on the command line.

CLI help should be updated as well to show libsql scheme and not credentials:

Usage:
  turso db shell {database_name | replica_url} [sql] [flags]

Examples:
turso db shell name-of-my-amazing-db
turso db shell https://<login>:<password>@<replica-url>
turso db shell https://alice:94l6z30w1Kq8p7ob@e784400f26d083-my-amazing-db-replica-url.turso.io

athoscouto commented 1 year ago

Not as trivial as I've thought, will have to prioritize other stuff for now.

haaawk commented 1 year ago

I'm taking this over as it breaks CLI tests so I need to fix it.

penberg commented 1 year ago

Fixed by cbcb29a8c6042ef17956a3e78d4067d38fa2fb80

haaawk commented 1 year ago

I think I didn't fix the help wording.

haaawk commented 1 year ago

It's fixed now.

tursodatabase / turso-cli

Shell needs to understand and show libsql URLs #290