Skip to content

feat(put multipart): add parsing for multipart content with methods other than POST#107

Open
dariok wants to merge 3 commits into
eeditiones:mainfrom
dariok:issue_104
Open

feat(put multipart): add parsing for multipart content with methods other than POST#107
dariok wants to merge 3 commits into
eeditiones:mainfrom
dariok:issue_104

Conversation

@dariok

@dariok dariok commented Feb 14, 2026

Copy link
Copy Markdown

As described in #104, eXist does not currently parse multipart content for anything but POST requests.
When PUTing multipart content, roaster will fail as it cannot find the required parameters.

This PR adds parsing of multipart content for other request types, creating a structure basically identical to that of eXist’s native parsing.

@dariok

dariok commented Jul 15, 2026

Copy link
Copy Markdown
Author

This patch is still necessary in eXist 7.0.0-Beta3 due to several problems in both eXist and roaster.

@dariok

dariok commented Jul 15, 2026

Copy link
Copy Markdown
Author

Actually, this doesn’t help any more as everything’s just too broken.

With eXist 7.0.0Beta3, if a multipart-upload has been made, request:get-data() will always be empty (checked in router.xql:90). This means, I cannot parse the data that has been uploaded.
I guess this is connected to issue 5030 (linked above) and because eXist has already tried to parse the multipart data.

This means, I cannot parse the data that has been uploaded and create a useful map.

But, as said, eXist doesn’t really do a good job of parsing multipart uploads.
I will open an issue for eXist and revert to eXist 6.4.1 for the time being where this patch still works.

@dariok

dariok commented Jul 16, 2026

Copy link
Copy Markdown
Author

To track the upstream issues in eXist, I opened new issues there:

When both are fixed, this PR will be obsolete.

As the request data are gone due to the attempt at parsing (cf. eXist-db/exist#1536 (comment)), there is not way to PUT data.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant