Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

v3.2: Explain Param/Header/Enc example serialization#4673

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
handrews wants to merge8 commits intoOAI:v3.2-devfromhandrews:style-examples-2

Conversation

handrews
Copy link
Member

@handrewshandrews commentedJun 9, 2025
edited by ralfhandl
Loading

NOTE: This is another PR split out from PR#4647, and makessignificant changes to thestyle serialization for headers compared to that PR, and clarifies the other policies that already existed in PR#4647. This should be more consistent than what was there before, as I commented in the last call when I was unsure of header serialization.

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.

  • 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

@handrewshandrews added this to thev3.2.0 milestoneJun 9, 2025
@handrewshandrews requested review froma team ascode ownersJune 9, 2025 18:29
@handrewshandrews added param serializationIssues related to parameter and/or header serialization example obj/keywordsIssues with the Example Object or exampel(s) keywords labelsJun 9, 2025
@handrews
Copy link
MemberAuthor

I have added updates to the Parameter and Header Object examples, as they fit here (and unlike with the Example Object, do not cause the diff to become unreadable).

If you would like more examples added, please help get this PR merged and then feel free to open further PRs. There are endless things we could add and I would like to get the existing things updated first.

@handrews
Copy link
MemberAuthor

I have also added a more complex header example, which@karenetheridge I think addresses your concern in this area.

src/oas.md Outdated
@@ -2471,6 +2513,42 @@ ETag:
pattern: ^"
```

A `Link` header that, if present, must include links with the standard relation types `self`, `first`, `prev`, `next`, and `last`, as might be used on a paginated collection:

Choose a reason for hiding this comment

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

I love this example!

handrews reacted with heart emoji
Copy link
MemberAuthor

Choose a reason for hiding this comment

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

@karenetheridge I have (hopefully) improved it in the latest commit, which has the message:


Actually ensure each pattern is matched - in the previous way
the example was written, the uniqueItems was not sufficient
as a link with the same rel but different URI would have satisfied
the conditions.

Also, the text says "includes" these rels, so there could be more
entries.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

Assuming theapplication/linkset PR gets accepted, this example is obsolete, but I can show a similar technique withSet-Cookie, which is also good because it has weird serialization requirements which we need to address for examples, which I will do. RFC9110 directs implementations to handleSet-Cookie as a special case, so I will follow that guidance and describe how to work with it rather than trying to change anything else to accommodate it. Stay tuned...

miqui
miqui previously approved these changesJun 12, 2025
The rules for this have not been clear, and are not always intuitive.This states and explains them directly and ensures that theStyle Examples table matches the rules.
These updates are about header usage in the Response Object.
Actually ensure each pattern is matched - in the previous waythe example was written, the uniqueItems was not sufficientas a link with the same rel but different URI would have satisfiedthe conditions.Also, the text says "includes" these rels, so there could be moreentries.
The `Link` header is now addressed in another PR.Make the first example a real header instead of an `X-`.Also provide a link to the Header Object for special casesthat will be documented there in other PRs.
@handrews
Copy link
MemberAuthor

Between the ongoing debate overserializedValue in the Example Object and the discussions here, I'm re-thinking how to explain this better. Putting this in draft for now- I might revive it or close it in favor of new PRs.

@handrews
Copy link
MemberAuthor

handrews commentedJul 18, 2025
edited
Loading

Closing in favor of#4800 and#4801.

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

@karenetheridgekarenetheridgekarenetheridge left review comments

@hudlowhudlowhudlow left review comments

@miquimiquimiqui left review comments

Assignees
No one assigned
Labels
example obj/keywordsIssues with the Example Object or exampel(s) keywordsparam 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@karenetheridge@hudlow@miqui

[8]ページ先頭

©2009-2025 Movatter.jp