Skip to content

v3.1.2: Explain Param and Header example serialization (backports PR 4800) #4826

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

Merged
merged 2 commits into from
Aug 12, 2025

Conversation

handrews
Copy link
Member

Backport of #4800, with the parts referencing the new Example Object fields and guidance on Set-Cookie removed. Note that we do have one use of the Example Object's value field for an application/x-www-form-urlencoded Media Type Object, and this change to the style examples table makes the table line up with the example. Which I think is a good thing. Since we're doing a bunch of other percent-encoding-related tidying, we might as well include this and make 3.1.2 and 3.2.0 consistent.

Commit message follows:


The rules for this have not been clear, and are not always intuitive. This states and explains them directly and ensures that the Style Examples table matches the rules.

Unlike past efforts, this provides a rule system regarding what is and is not included, based on a combination of what is produced by RFC6570 (or the nearest RFC6570 equivalent), modified by removing leading delimiters that are not correct for our usage due to differences from the assuptions made by RFC6570.

  • schema changes are included in this pull request
  • schema changes are needed for this pull request but not done yet
  • no schema changes are needed for this pull request

The rules for this have not been clear, and are not always intuitive.
This states and explains them directly and ensures that the
Style Examples table matches the rules.

Unlike past efforts, this provides a rule system regarding what
is and is not included, based on a combination of what is produced
by RFC6570 (or the nearest RFC6570 equivalent), modified by
removing leading delimiters that are not correct for our usage
due to differences from the assuptions made by RFC6570.
@handrews handrews added this to the v3.1.2 milestone Aug 10, 2025
@handrews handrews added the clarification requests to clarify, but not change, part of the spec label Aug 10, 2025
@handrews handrews requested review from a team as code owners August 10, 2025 21:16
@handrews handrews added 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) approved pr port PRs that just port an approved PR to another version labels Aug 10, 2025
@handrews handrews changed the title Explain Param and Header example serialization v3.1.2: Explain Param and Header example serialization (backports PR 4800) Aug 10, 2025
miqui
miqui previously approved these changes Aug 10, 2025
Copy link
Contributor

@ralfhandl ralfhandl left a comment

Choose a reason for hiding this comment

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

+1 in general, please remove dataValue

@ralfhandl ralfhandl requested review from miqui and a team August 11, 2025 18:09
Copy link
Contributor

@lornajane lornajane left a comment

Choose a reason for hiding this comment

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

Thanks!

@lornajane lornajane merged commit 786284f into OAI:v3.1-dev Aug 12, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved pr port PRs that just port an approved PR to another version clarification requests to clarify, but not change, part of the spec media and encoding Issues regarding media type support and how to encode data (outside of query/path params) param serialization Issues related to parameter and/or header serialization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants