Skip to content

Fix generated openapi.json parameter names - #600

Merged
rbygrave merged 2 commits into
masterfrom
feature/fix-openapi-json-parameter-names
Apr 29, 2025
Merged

Fix generated openapi.json parameter names#600
rbygrave merged 2 commits into
masterfrom
feature/fix-openapi-json-parameter-names

Conversation

@rbygrave

@rbygrave rbygrave commented Apr 29, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor 
@QueryParam String lastName               // -> "lastName" (no bug here, this was OK)
@QueryParam("q-2") String param2          // -> "q-2"
@Header String contentLength              // -> "Content-Length" (because headers automatic names are Init-Camel)
@Header("x-oh") String otherHeader        // -> "x-on"

@rbygrave
rbygrave requested a review from SentryMan April 29, 2025 07:17
@rbygrave rbygrave self-assigned this Apr 29, 2025
@rbygrave

rbygrave commented Apr 29, 2025

Copy link
Copy Markdown
Contributor Author

Ok, I'll merge this in before a PR review because I'm wanting another RC release to test out. If you are not liking this change @SentryMan we can adjust again.

@rbygrave
rbygrave merged commit c25de39 into master Apr 29, 2025
@rbygrave
rbygrave deleted the feature/fix-openapi-json-parameter-names branch April 29, 2025 12:17
@SentryMan SentryMan added this to the 3.4 milestone Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SentryMan SentryMan Awaiting requested review from SentryMan

Assignees

@rbygrave rbygrave

Labels

None yet

Projects

None yet

Milestone

3.4

Development

Successfully merging this pull request may close these issues.

2 participants

@rbygrave @SentryMan