diff options
| author | Vladimir Kochnev <hashtable@yandex.ru> | 2022-08-26 03:24:40 +0300 |
|---|---|---|
| committer | Vladimir Kochnev <hashtable@yandex.ru> | 2022-08-26 09:55:17 +0300 |
| commit | cfdca829382b83081a897ae36c4196678b52a9b4 (patch) | |
| tree | 55b7fd9e9084307aad138ea8148c62d1e70c1bb3 /pkg/bindings/internal/util | |
| parent | 36cf6f572a360af91bf261936b6ed6849f355478 (diff) | |
| download | podman-cfdca829382b83081a897ae36c4196678b52a9b4.tar.gz podman-cfdca829382b83081a897ae36c4196678b52a9b4.tar.bz2 podman-cfdca829382b83081a897ae36c4196678b52a9b4.zip | |
Mark some of the option fields as ignored in pkg/bindings
I realized that `params.Del("SkipTLSVerify")` doesn't have any
effect because keys are always lowercased. So it should really
be `params.Del("skiptlsverify")`.
There's also a little bug introduced by 3bf52aa and b1d1248: if
one passes `ProgressWriter` object having `Stringer` interface
i.e. `bytes.Buffer` it ends up been serialized in query with
`util.ToParams()`.
To circumvent both problems I propose to mark non-serializable
parameters with `schema:"-"` so there's no need to delete them from
resulting `url.Values`.
Signed-off-by: Vladimir Kochnev <hashtable@yandex.ru>
Diffstat (limited to 'pkg/bindings/internal/util')
| -rw-r--r-- | pkg/bindings/internal/util/util.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/pkg/bindings/internal/util/util.go b/pkg/bindings/internal/util/util.go index f8f99d6c1..52ce14738 100644 --- a/pkg/bindings/internal/util/util.go +++ b/pkg/bindings/internal/util/util.go @@ -74,6 +74,9 @@ func ToParams(o interface{}) (url.Values, error) { } paramName := fieldName if pn, ok := sType.Field(i).Tag.Lookup("schema"); ok { + if pn == "-" { + continue + } paramName = pn } switch { |