1berry/docker-cli
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

cli/connhelper/ssh: tweak error-message (capitalize SSH)

Browse Source 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This commit is contained in:
Sebastiaan van Stijn 2025-04-23 14:27:49 +02:00
parent 81a5db6b82
commit 55073c404c
No known key found for this signature in database
GPG Key ID: 76698F39D527CE8C
2 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cli/connhelper/ssh/ssh.go
View File

@ -17,11 +17,11 @@ func ParseURL(daemonURL string) (*Spec, error) {
if errors.As(err, &urlErr) { if errors.As(err, &urlErr) {
err = urlErr.Unwrap() err = urlErr.Unwrap()
} }
return nil, fmt.Errorf("invalid ssh URL: %w", err) return nil, fmt.Errorf("invalid SSH URL: %w", err)
} }
s, err := newSpec(u) s, err := newSpec(u)
if err != nil { if err != nil {
return nil, fmt.Errorf("invalid ssh URL: %w", err) return nil, fmt.Errorf("invalid SSH URL: %w", err)
} }
return s, nil return s, nil
} }

16
cli/connhelper/ssh/ssh_test.go
View File

@ -87,41 +87,41 @@ func TestParseURL(t *testing.T) {
{ {
doc: "malformed URL", doc: "malformed URL",
url: "malformed %%url", url: "malformed %%url",
expectedError: `invalid ssh URL: invalid URL escape "%%u"`, expectedError: `invalid SSH URL: invalid URL escape "%%u"`,
}, },
{ {
doc: "URL missing scheme", doc: "URL missing scheme",
url: "no-scheme.example.com", url: "no-scheme.example.com",
expectedError: "invalid ssh URL: no scheme provided", expectedError: "invalid SSH URL: no scheme provided",
}, },
{ {
doc: "invalid URL with password", doc: "invalid URL with password",
url: "ssh://me:passw0rd@example.com", url: "ssh://me:passw0rd@example.com",
expectedError: "invalid ssh URL: plain-text password is not supported", expectedError: "invalid SSH URL: plain-text password is not supported",
}, },
{ {
doc: "invalid URL with query parameter", doc: "invalid URL with query parameter",
url: "ssh://example.com?foo=bar&bar=baz", url: "ssh://example.com?foo=bar&bar=baz",
expectedError: `invalid ssh URL: query parameters are not allowed: "foo=bar&bar=baz"`, expectedError: `invalid SSH URL: query parameters are not allowed: "foo=bar&bar=baz"`,
}, },
{ {
doc: "invalid URL with fragment", doc: "invalid URL with fragment",
url: "ssh://example.com#bar", url: "ssh://example.com#bar",
expectedError: `invalid ssh URL: fragments are not allowed: "bar"`, expectedError: `invalid SSH URL: fragments are not allowed: "bar"`,
}, },
{ {
doc: "invalid URL without hostname", doc: "invalid URL without hostname",
url: "ssh://", url: "ssh://",
expectedError: "invalid ssh URL: hostname is empty", expectedError: "invalid SSH URL: hostname is empty",
}, },
{ {
url: "ssh:///no-hostname", url: "ssh:///no-hostname",
expectedError: "invalid ssh URL: hostname is empty", expectedError: "invalid SSH URL: hostname is empty",
}, },
{ {
doc: "invalid URL with unsupported scheme", doc: "invalid URL with unsupported scheme",
url: "https://example.com", url: "https://example.com",
expectedError: `invalid ssh URL: incorrect scheme: https`, expectedError: `invalid SSH URL: incorrect scheme: https`,
}, },
} }
for _, tc := range testCases { for _, tc := range testCases {