X Tutup
Skip to content

Fix/issue 2261#2262

Open
mromaszewicz wants to merge 1 commit intooapi-codegen:mainfrom
mromaszewicz:fix/issue-2261
Open

Fix/issue 2261#2262
mromaszewicz wants to merge 1 commit intooapi-codegen:mainfrom
mromaszewicz:fix/issue-2261

Conversation

@mromaszewicz
Copy link
Member

No description provided.

@mromaszewicz mromaszewicz requested a review from a team as a code owner February 28, 2026 15:50
@mromaszewicz mromaszewicz added the bug Something isn't working label Feb 28, 2026
…r RequestObject

ParameterDefinition.JsonTag() only produced a `json:"..."` tag and
ignored x-oapi-codegen-extra-tags. This meant path parameters in
strict-server RequestObject structs never included extra struct tags,
even though query/header/cookie parameters did (via GenerateParamsTypes).

Update JsonTag() to read x-oapi-codegen-extra-tags from both the
parameter and schema levels, with parameter-level taking precedence,
matching the existing merge behavior in GenerateParamsTypes().

Fixes oapi-codegen#2261

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

x-oapi-codegen-extra-tags on path parameters not generated in strict-server RequestObject structs

1 participant

X Tutup