Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0x1F as nested namespace separator for REST URL paths conflicts with Servlet spec #10338

Open
westse opened this issue May 15, 2024 · 0 comments
Labels
improvement PR that improves existing functionality

Comments

@westse
Copy link
Contributor

westse commented May 15, 2024

Feature Request / Improvement

The current Iceberg REST spec uses 0x1F to separate the parts of a multipart (nested) namespace parameter in URL paths and elsewhere. However this conflicts with the Servlet spec which calls for rejecting control characters (encoded or not), including the unit separator (0x1F), as "suspicious" and to return 400. Servlet containers enforce this (e.g. see jetty/jetty.project#11495), making implementing the Iceberg REST spec difficult.

Should a different, or at least alternative, separator character be selected to avoid conflicting with the Servlet spec?

Query engine

None

@westse westse added the improvement PR that improves existing functionality label May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement PR that improves existing functionality
Projects
None yet
Development

No branches or pull requests

1 participant