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

Support more and variable media types, add rawBody per claudiajs documentation #43

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

hibes
Copy link

@hibes hibes commented Jan 7, 2022

The goal of this PR is to get a little closer to being in-spec with the ClaudiaJS documentations surrounding the body and rawBody functionality.

Specifically, I noticed that if I try to either upload a file using a curl POST request or if I try to upload a string while specifying any mimetype other than application/json or application/x-www-form-urlencoded that the body is not correct. It will always have the empty object value ({}) and never have the contents which I attempted to submit via POST.

Relevant ClaudiaJS docs:
https://github.com/claudiajs/claudia-api-builder/blob/master/docs/request-object.md

@hibes
Copy link
Author

hibes commented Jan 12, 2022

@suddi, I don't think I have permission to add you as a reviewer to this PR. Any chance you have some time to take a look at it and provide 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.

1 participant