-
Notifications
You must be signed in to change notification settings - Fork 337
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for application/x-www-form-urlencoded Content-Type as body type formdata currently only supports multipart/form-data #337
Labels
Comments
AcousticDeveloper
changed the title
Support for application/x-www-url-encoded Content-Type as body type formdata currently only supports multipart/form-data
Support for application/x-www-form-urlencoded Content-Type as body type formdata currently only supports multipart/form-data
Mar 20, 2024
animator
added
enhancement
New feature or request
good first issue
Good for newcomers
and removed
bug
Something isn't working
labels
Mar 20, 2024
4 tasks
A new content-type needs to be added but in UI it can be done in two ways:
|
@ashitaprasad I would like to give it a try |
Sure @himanshugoyal77 you can give it a try. It will be assigned once you send across a PR. |
Without this can't use it for getting access tokens for OAuth. |
@ilovefreesw You can use this for getting access tokens for OAuth |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug/problem
Currently whenever the formdata is selected as request body, the
Content-Type
header is set tomultipart/form-data
but this should be dependent upon whether there is a file being uploaded too or else it can be set to the defaultapplication/x-www-form-urlencoded
.Relevant References
stackoverflow: application/x-www-form-urlencoded or multipart/form-data?
baeldung: Difference Between form-data, x-www-form-urlencoded and raw in Postman
Steps to Reproduce the bug/problem
Select POST request in the request method, select formdata as the request body, add only text fields as fields, and then we can either chck the codegen for languages where Content-Type is printed, or else we can print
requestModel.requestBodyContentType.header
inside codegen files to check the request header.Expected behavior
If there aren't any files uploaded i.e. all the fields are of text type and selected type is formdata, then the
Content-Type
should be set toapplication/x-www-form-urlencoded
, and if there is at least one file in the form data, then theContent-Type
should bemultipart/form-data
. As the defaultContent-Type
for HTML form submission isapplication/x-www-form-urlencoded
, hence it is worth introducing in API dash.Reference: Mozilla Documentation
Fixing this should start from introducing a new sub-type inside
consts.dart
file and then creating a newContentType
for the new type and setting up the Content Type based on whether there are any files associated with the request or not. Below are some code snippets fromconsts.dart
which are to be modified to start working on this issue.Device Info (The device where you encountered this issue):
The text was updated successfully, but these errors were encountered: