Skip to content

Accept serialized array and object path parameters - #491

Merged
geemus merged 1 commit into
masterfrom
fix-serialized-path-parameters
Oct 20, 2026
Merged

geemus merged 1 commit into
masterfrom
fix-serialized-path-parameters

Conversation

@ydah

@ydah ydah commented Oct 10, 2026

Copy link
Copy Markdown
Member

OpenAPI 3 path arrays and objects serialized with simple, label, or matrix style are validated while still strings. For example, /paths/1,2 is rejected as a String instead of being validated as [1,2]. Deserialization happens later, and exploded matrix arrays remain one raw string.

Deserialize path values before passing them to openapi_parser for validation and coercion, including parameters declared on path items. Parse repeated matrix segments such as ;ids=1;ids=2 into separate elements.

@geemus

geemus commented Oct 17, 2026

Copy link
Copy Markdown
Member

@ydah could you resolve the conflicts on this one? The test one seems clear enough, but I'm feeling less confident on properly reconciling the validator. Thanks.

@ydah
ydah force-pushed the fix-serialized-path-parameters branch from d65b5c2 to c588eda Compare October 20, 2026 09:13
@geemus

geemus commented Oct 20, 2026

Copy link
Copy Markdown
Member

@ydah thanks, I was trying to work through it, but my brain was rebelling, so I thought I would ask you just in case. Much appreciated.

@geemus geemus left a comment

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.

lgtm, thanks!

@geemus
geemus merged commit 2beb521 into master Oct 20, 2026
9 checks passed
@geemus
geemus deleted the fix-serialized-path-parameters branch October 20, 2026 13:14
@ydah

ydah commented Oct 20, 2026

Copy link
Copy Markdown
Member Author

@geemus Thank you!

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