Skip to content

Conversation

iandyh
Copy link

@iandyh iandyh commented Jul 29, 2025

…ly handled

I was facing this issue with [email protected].

For a PUT request expecting application/x-www-form-urlencoded, The generated swagger.json has the correct config:

"consumes": ["application/x-www-form-urlencoded"]

However, the generated code incorrectly set the request to be sent with multipart/form-data.

type: ContentType.FormData

fyi: the code is entirely generated by Github Copilot.

Copy link

changeset-bot bot commented Jul 29, 2025

🦋 Changeset detected

Latest commit: 24704ea

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
swagger-typescript-api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@iandyh iandyh force-pushed the main branch 4 times, most recently from f5d92b7 to 9b8d877 Compare July 30, 2025 11:00
@smorimoto

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@iandyh
Copy link
Author

iandyh commented Aug 21, 2025

@smorimoto thanks for checking. I need further verification of the change since I notice something suspicious with this fix. I'll update you once I finish the confirmation.

@iandyh
Copy link
Author

iandyh commented Aug 22, 2025

@smorimoto I've verified the changes and they are good to go.

@smorimoto
Copy link
Collaborator

@codex review

Copy link

Codex Review: Didn't find any major issues. Breezy!

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

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