Commit ce5c65f8 authored by Nick Thomas's avatar Nick Thomas

Merge branch 'doc-graphql-uploads-issues' into 'master'

Document the pitfalls of GraphQL uploads

See merge request gitlab-org/gitlab!47494
parents 1a578046 68d090ca
...@@ -278,8 +278,10 @@ Uploads routes belong to one of these categories: ...@@ -278,8 +278,10 @@ Uploads routes belong to one of these categories:
1. Rails controllers: uploads handled by Rails controllers. 1. Rails controllers: uploads handled by Rails controllers.
1. Grape API: uploads handled by a Grape API endpoint. 1. Grape API: uploads handled by a Grape API endpoint.
1. GraphQL API: uploads handled by a GraphQL resolve function. In these cases, there is nothing else 1. GraphQL API: uploads handled by a GraphQL resolve function.
to do apart from implementing the actual upload.
CAUTION: **Warning:**
GraphQL uploads do not support [direct upload](#direct-upload) yet. Depending on the use case, the feature may not work on installations without NFS (like GitLab.com or Kubernetes installations). Uploading to object storage inside the GraphQL resolve function may result in timeout errors. For more details please follow [issue #280819](https://gitlab.com/gitlab-org/gitlab/-/issues/280819).
### Update Workhorse for the new route ### Update Workhorse for the new route
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment