Skip to content

v3.3: use extra markup to clarify intent#5331

Open
karenetheridge wants to merge 1 commit into
OAI:v3.3-devfrom
karenetheridge:ether/v3.3-encoding-markup
Open

v3.3: use extra markup to clarify intent#5331
karenetheridge wants to merge 1 commit into
OAI:v3.3-devfrom
karenetheridge:ether/v3.3-encoding-markup

Conversation

@karenetheridge
Copy link
Copy Markdown
Member

We are specifically talking about the "encoding" keyword here, and the multipart/form-data media-type (most other multipart types decode to arrays).

  • no schema changes are needed for this pull request

@karenetheridge karenetheridge requested review from a team as code owners May 13, 2026 19:55
We are specifically talking about the "encoding" keyword here, and the
multipart/form-data media-type (most other multipart types decode to arrays).
@karenetheridge karenetheridge force-pushed the ether/v3.3-encoding-markup branch from 4398f00 to 17845bf Compare May 13, 2026 19:55
@karenetheridge karenetheridge changed the title use extra markup to clarify intent v3.3: use extra markup to clarify intent May 13, 2026
Comment thread src/oas.md
See [Encoding the `x-www-form-urlencoded` Media Type](#encoding-the-x-www-form-urlencoded-media-type) for guidance and examples, both with and without the `encoding` field.

For `multipart`, the encoding keys MUST map to the [`name` parameter](https://www.rfc-editor.org/rfc/rfc7578#section-4.2) of the `Content-Disposition: form-data` header of each part, as is defined for `multipart/form-data` in [[!RFC7578]].
For `multipart/form-data`, the `encoding` keys MUST map to the [`name` parameter](https://www.rfc-editor.org/rfc/rfc7578#section-4.2) of the `Content-Disposition: form-data` header of each part, as is defined for `multipart/form-data` in [[!RFC7578]].
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@karenetheridge I agree with you in general here, but there's a bit of history.

The reason this does not say /form-data is because of the paragraph further down along the lines of "MAY use a name parameter with other multipart but most multipart implementations probably won't support it."

I'm unsure what to do here. That part was added as a possible workaround for lack of proper sequential multipart support, and I somewhat regret it as now it's this weird thing that makes it harder to make clear statements like this.

We could say "For multipart/form-data (or any multipart format making use of a name paraameter)..."?

Copy link
Copy Markdown
Member Author

@karenetheridge karenetheridge May 13, 2026

Choose a reason for hiding this comment

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

How about "For multipart types that decode to an object... such as multipart/form-data in RFC7578" ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think that works. In theory there could be some other multipart that encodes to an object by some other mechanism, but... I have never heard of such a thing and I've looked, so this should be fine.

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.

2 participants