Skip to content

fix(rest): improve REST catalog configuration and error handling - #1003

Open
lishuxu wants to merge 1 commit into
apache:mainfrom
lishuxu:feature/dev-1
Open

fix(rest): improve REST catalog configuration and error handling#1003
lishuxu wants to merge 1 commit into
apache:mainfrom
lishuxu:feature/dev-1

Conversation

@lishuxu

@lishuxu lishuxu commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

This commit improves the robustness of REST catalog operations:

  • Disable HTTP redirects in REST catalog requests to prevent unexpected behavior with signed requests and custom authentication headers
  • Filter server-provided configuration to prevent override of client-specified authentication settings
  • Improve JSON parse error messages to avoid exposing sensitive data in logs by reporting only type information instead of full values
  • Add log sanitizer utility to redact sensitive configuration values from application logs

@lishuxu
lishuxu force-pushed the feature/dev-1 branch 4 times, most recently from cd04e91 to 01faa9a Compare October 15, 2026 09:55
shuxu.li
fix(json): do not include JSON values in parse error messages
5b33eec 
Parse errors from the JSON helpers and the table metadata and REST
response parsers interpolated the offending JSON document, and
FromJsonString echoed the whole input on invalid JSON. Report the key,
the JSON type found and the reason instead, and report only the input
size for invalid JSON.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@lishuxu