File upload mutation with Abinthe giving "no query document supplied" error

Yes, URQL is using a different pattern for file uploads. I’ve been fighting this spec forever because I hate the use of null as a marker for a value that is not null. However it seems this spec is getting popular and this is a fight I’m going to lose eventually. sigh.

I see. I agree with your thinking. It seems like URQL itself is pretty modular and I’m using the @urql/exchange-multipart-fetch package (which is part of the core repo) to provide this multipart functionality.

I guess it shouldn’t be too hard to make an Absinthe-specific equivilent.

It would be nice if there was agreement on a standard spec though.

There does seem to be a standard for this: GitHub - jaydenseric/graphql-multipart-request-spec: A spec for GraphQL multipart form requests (file uploads). · GitHub. Although I have no idea who has the final say on these things.

In any case, I managed to make a exchange for URQL that uses the pattern adopted by Absinthe.

Yes this is exactly the spec I’m talking about. To be clear, this is the proposed spec by one person. I grant that it has seen increasing adoption within the JS graphql ecosystem, but it is not an official spec by the GraphQL body.

You can read about my specific objects to that spec here: Null means the opposite of Null · Issue #4 · jaydenseric/graphql-multipart-request-spec · GitHub

I found this thread when googling. I didn’t want to rewrite anything on the client because that sounds pretty rough, so I went for the Elixir rewriting approach. I hooked into multipart_to_params, an option to the Plug.Parsers plug that’s usually put in Endpoint

This code works for my simple case locally. I didn’t test with multiple files because that’s not a use case for me. I also didn’t worry about graceful params handling. Use at own risk, but possibly a decent starting point:

defmodule RewriteMultipart do
  def multipart_params(parts, conn) do
    params =
      for {name, _headers, body} <- parts,
          name != nil,
          reduce: %{} do
        acc -> Plug.Conn.Query.decode_pair({name, body}, acc)
      end
​
    params = case params do
      %{"operations" => operations, "map" => map} ->
        map = Jason.decode!(map)
        operations = Jason.decode!(operations)
​
        # Transform the file map to the query variables
        vars =
          Enum.reduce(map, operations["variables"], fn mapping, vars ->
            {name, [path]} = mapping
            path = String.split(path, ".") -- ["variables"]
            put_in(vars, path, name)
          end)
​
        # New query drops operations/map in favor of query/variables
        params
        |> Map.drop(["operations", "map"])
        |> Map.merge(%{"variables" => vars, "query" => operations["query"]})
​
      _ ->
        params
    end
​
    {:ok, params, conn}
  end
end

for posterity, i adapted some solutions into this with some basic tests and a way to do an integration test absinthe integration with apollo's file upload · GitHub