Skip to content

v3.2: Port of percent-encoding-related fixes from 3.1.2 - #4863

Merged
lornajane merged 6 commits into
OAI:v3.2-devfrom
handrews:port-pct
Aug 14, 2025
Merged

v3.2: Port of percent-encoding-related fixes from 3.1.2#4863
lornajane merged 6 commits into
OAI:v3.2-devfrom
handrews:port-pct

Conversation

@handrews

@handrewshandrews commented Aug 13, 2025•
edited by karenetheridge
Loading

Copy link
Copy Markdown
Member

This ports the following PRs, each in their own commit (@ralfhandl this was easily done because of the lack of interspersed merge commits 🙂 ), in the following order:

Once this is merged, I can finish the style: cookie PR, and fix allowReserved to only apply to in values of path and query, and do some other related 3.2-specific tweaks that I did not want to mix with the approved PRs.

  • no schema changes are needed for this pull request

@handrewshandrews added this to the v3.2.0 milestone Aug 13, 2025
@handrews
handrews requested review from a team as code ownersAugust 13, 2025 16:45
@handrewshandrews added clarification requests to clarify, but not change, part of the spec param serialization Issues related to parameter and/or header serialization media and encoding Issues regarding media type support and how to encode data (outside of query/path params) headers approved pr port PRs that just port an approved PR to another version labels Aug 13, 2025
@handrews
New percent-encoding section under Parameter Object
d50527b
@handrews
Fix guidance for RFC6570 and multipart/form-data
1264d08
Research has determined that percent-encoding was never intended
to apply to this media type.
@handrews
Fix guidance on headers and RFC6570 percent-encoding
0fd14b6
After much debate and research, we agreed that percent-encoding
was never meant to be applied to headers.
Exactly how to handle RFC6570 and cookie parameters remains TBD.
For now, this preserves (but streamlines) the existing guidance
for cookies.
@handrews
Fix list formatting (use periods at end)
56faefa
@handrews
Update examples and appendicies for percent-encoding
f100052
After adding a new section on percent-encoding guidance, this
updates the examples and other supplemental text to match it.
@handrewshandrews mentioned this pull request Aug 13, 2025
Closed
ralfhandl
ralfhandl previously approved these changes Aug 13, 2025
View reviewed changes
@ralfhandl
ralfhandl requested a review from a teamAugust 13, 2025 20:30
lornajane
lornajane approved these changes Aug 14, 2025
View reviewed changes

@lornajanelornajane 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.

Thank you

@lornajane
lornajane merged commit 2a86e81 into OAI:v3.2-devAug 14, 2025
2 checks passed
@lornajanelornajane mentioned this pull request Aug 17, 2025
Closed
@handrews
handrews deleted the port-pct branch August 27, 2025 17:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lornajanelornajanelornajane approved these changes
@miquimiquimiqui approved these changes
@ralfhandlralfhandlralfhandl left review comments

Assignees

No one assigned

Labels

approved pr portPRs that just port an approved PR to another versionclarificationrequests to clarify, but not change, part of the specheadersmedia and encodingIssues regarding media type support and how to encode data (outside of query/path params)param serializationIssues related to parameter and/or header serialization

Projects

None yet

Milestone

v3.2.0

Development

Successfully merging this pull request may close these issues.

4 participants

@handrews@lornajane@ralfhandl@miqui