-
Notifications
You must be signed in to change notification settings - Fork 135
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
Handle mutlipart/form-data #43
base: master
Are you sure you want to change the base?
Conversation
@chris-ramon Nudge - would it be possible to have this looked at? |
hi all, When will it merge to master ? |
@chris-ramon Any chance that this could get a review, will clean up the conflict shortly. |
@koblas conflicts |
@appleboy Conflicts resolved... Hopefully, someday @chris-ramon will feel like merging. |
Can we merge this? |
While waiting this being merged, found an alternative handler supporting Originally posted at graphql-go/graphql#607 Just provide one more option 😃 |
This PR should support Multipart form-data file upload requests as specified by https://github.com/jaydenseric/graphql-multipart-request-spec -- which is adopted by the Apollo team. It also works without file uploads as a general handler.