Skip to content

Commit 9d1dcb3

Browse files
committed
review fixes
1 parent b87a7fb commit 9d1dcb3

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

framework/components/clnode/clnode.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ func generatePortBindings(in *Input) ([]string, nat.PortMap, error) {
158158
if strings.Contains(p, CustomPortSeparator) {
159159
pp := strings.Split(p, CustomPortSeparator)
160160
if len(pp) != 2 {
161-
return nil, nil, errors.New("custom_ports has ':' but you must provide both ports")
161+
return nil, nil, fmt.Errorf("custom_ports has ':' but you must provide both ports, you provided: %s", pp)
162162
}
163163
customPorts = append(customPorts, fmt.Sprintf("%s/tcp", pp[1]))
164164

framework/docker.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import (
66
"bytes"
77
"context"
88
"encoding/binary"
9-
"errors"
109
"fmt"
1110
"io"
1211
"os"
@@ -425,7 +424,7 @@ func GenerateCustomPortsData(portsProvided []string) ([]string, nat.PortMap, err
425424
if strings.Contains(p, ":") {
426425
pp := strings.Split(p, ":")
427426
if len(pp) != 2 {
428-
return nil, nil, errors.New("custom_ports has ':' but you must provide both ports")
427+
return nil, nil, fmt.Errorf("custom_ports has ':' but you must provide both ports, you provided: %s", pp)
429428
}
430429
customPorts = append(customPorts, fmt.Sprintf("%s/tcp", pp[1]))
431430

0 commit comments

Comments
 (0)