richiebachala commented on PR #2190:
URL: https://github.com/apache/iceberg-go/pull/2190#issuecomment-6241084623
A correction to my last comment: I said `--fail-with-body` makes a 401 show
its body. It didn't. The script piped the token body into `jq` and sent the
other bodies to `/dev/null`, so a failure only printed `curl: (22) ... error:
401`.
Fixed in 637fc58: API calls now go through a small helper that prints the
server's error body on failure. Forcing a bad secret now shows
`{"error":"unauthorized_client",...}`, and a duplicate catalog shows Polaris's
`Catalog already exists` message. Same commit: the self-rename log now prints
the actual error instead of a fixed message, and the script uses 2-space
indents like the repo's other scripts.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
