Skip to content
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

Data mapping for application/www-x-form-urlencoded #2526

Closed
shafreenAnfar opened this issue Dec 21, 2021 · 3 comments · Fixed by ballerina-platform/module-ballerina-http#823
Assignees
Labels
module/http Team/PCM Protocol connector packages related issues Type/Improvement

Comments

@shafreenAnfar
Copy link
Contributor

shafreenAnfar commented Dec 21, 2021

At the moment we do the data mapping as in #976. But we have missed the below data mapping.

Type Media Type
map<string> application/www-x-form-urlencoded
@shafreenAnfar shafreenAnfar added Type/Improvement module/http Team/PCM Protocol connector packages related issues labels Dec 21, 2021
@shafreenAnfar
Copy link
Contributor Author

I think this a blocker for the release because we have missed a common content-type.

@chamil321
Copy link
Contributor

@ayeshLK Let's add the map type support for the service binding first. Then gradually introduce it to the client. When introducing map<string> if the content type of the annotation is @http:Payload{mediaType : "application/www-x-form-urlencoded"} , get the payload as string and create map. Otherwise, get the payload as json

@shafreenAnfar
Copy link
Contributor Author

Here is the proposal #2530

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module/http Team/PCM Protocol connector packages related issues Type/Improvement
Projects
None yet
3 participants