diff options
| author | Chris Evich <cevich@redhat.com> | 2022-07-13 14:41:16 -0400 |
|---|---|---|
| committer | Matthew Heon <matthew.heon@pm.me> | 2022-07-26 13:26:12 -0400 |
| commit | 3bd248ae7cd5ebaaa1574f347729ce7839b5afe1 (patch) | |
| tree | 8e209e51f445ab17fc4597f0576c0c54fe3de642 /LICENSE | |
| parent | 6d84a9952f1e5be1a187bcc6d9bbc2532331cfc8 (diff) | |
| download | podman-3bd248ae7cd5ebaaa1574f347729ce7839b5afe1.tar.gz podman-3bd248ae7cd5ebaaa1574f347729ce7839b5afe1.tar.bz2 podman-3bd248ae7cd5ebaaa1574f347729ce7839b5afe1.zip | |
GHA: Fix dumb error check
Previously the reply JSON was examined for the literal presence of the
string 'error'. This was intended to catch server or query errors and
the like. However it's not a sound design as valid/legitimate contents
could potentially contain the string. Fix this by using the `-e` option
to `jq`, with a filter that should always result in a non-empty/null
match. If this fails or returns null for some reason, then it's safe to
throw a real error code & message.
Signed-off-by: Chris Evich <cevich@redhat.com>
Diffstat (limited to 'LICENSE')
0 files changed, 0 insertions, 0 deletions