diff options
| author | Daniel J Walsh <dwalsh@redhat.com> | 2020-07-29 14:10:59 -0400 |
|---|---|---|
| committer | Daniel J Walsh <dwalsh@redhat.com> | 2020-07-29 14:12:19 -0400 |
| commit | 9917fc0f956c849ec2d52f3f873aa6eaa3f25e72 (patch) | |
| tree | 4ca3031ca76b8664a29841e43b5e0271c14f933e /cmd/podman/system | |
| parent | 7f38774ee7758e56cf31e87c81ae0bc9208d0ede (diff) | |
| download | podman-9917fc0f956c849ec2d52f3f873aa6eaa3f25e72.tar.gz podman-9917fc0f956c849ec2d52f3f873aa6eaa3f25e72.tar.bz2 podman-9917fc0f956c849ec2d52f3f873aa6eaa3f25e72.zip | |
Don't crash when giving bogus format commands
Currently if you give a bogus flag to --format it will crash
the formatter. With this change we will get a nice error.
podman images --format '{{ bogus }}'
Error: template: list:1: function "bogus" not defined
versus
/bin/podman.old images --format '{{ bogus }}'
panic: template: list:1: function "bogus" not defined
goroutine 1 [running]:
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'cmd/podman/system')
| -rw-r--r-- | cmd/podman/system/connection/list.go | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/cmd/podman/system/connection/list.go b/cmd/podman/system/connection/list.go index 6d3d85d11..9010ec803 100644 --- a/cmd/podman/system/connection/list.go +++ b/cmd/podman/system/connection/list.go @@ -75,7 +75,11 @@ func list(_ *cobra.Command, _ []string) error { // TODO: Allow user to override format format := "{{range . }}{{.Name}}\t{{.Identity}}\t{{.URI}}\n{{end}}" - tmpl := template.Must(template.New("connection").Parse(format)) + tmpl, err := template.New("connection").Parse(format) + if err != nil { + return err + } + w := tabwriter.NewWriter(os.Stdout, 8, 2, 2, ' ', 0) defer w.Flush() |