Skip to content

Update multipart.lua#40

Open
xqpmjh wants to merge 1 commit into
Kong:masterfrom
xqpmjh:fix-boundary-parse
Open

Update multipart.lua#40
xqpmjh wants to merge 1 commit into
Kong:masterfrom
xqpmjh:fix-boundary-parse

Conversation

@xqpmjh

@xqpmjh xqpmjh commented Apr 18, 2023

Copy link
Copy Markdown

Payload:
multipart/form-data; boundary=6uPJdiTx1BvOyBOcFbt2eQN7tuBo3ZaYUmbFxoh; charset=UTF-8

This will add ";" to the end of the boundary:
local boundary = match(content_type, ";%s*boundary=(%S+)")

Payload:
multipart/form-data; boundary=6uPJdiTx1BvOyBOcFbt2eQN7tuBo3ZaYUmbFxoh; charset=UTF-8

This will add ";" to the end of the boundary:
local boundary = match(content_type, ";%s*boundary=(%S+)")
@CLAassistant

CLAassistant commented Apr 18, 2023

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Tieske Tieske left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense. Can you add a test case please?

@Tieske

Tieske commented Aug 31, 2023

Copy link
Copy Markdown
Contributor

ping @xqpmjh

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.

3 participants