Not possible to read HttpResponseMessage in case of AggregateException

[rlipovski]

I'm having a hard time to read the Content from the HttpResponseMessage inside the GraphQLHttpException. The code is throwing AggregateException, which holds the InnerExceptions property of type GraphQLHttpException. When I try to read the Content to figure out why my request is not working .net says the object has been disposed respectively, which is also ok as the client is implemented as a singleton. Why singleton? As far as I understood tthe graphql-client is a wrapper around HttpClient, which should be instantiated as singletons due to some internal bug(s).

This is what I tried, but no results:

        try
        {

            var request = new GraphQLRequest()
            {
                Query = CreateQuery(query)
            };


            client.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Bearer","token_goes_here");
            client.Options = new GraphQLClientOptions()
            {
                MediaType = new MediaTypeHeaderValue("application/json"),
                EndPoint = new Uri(listingImagesGraphqlUrl, UriKind.Absolute)
            };
            var result = client.PostAsync(request).Result;

            if (result.Errors.Any())
            {
                var unprocessedIds = result.Errors.Select(x => x.AdditonalEntries)
                    .Where(y => y.ContainsKey("extensions")).Where(z => z.ContainsKey("unprocessedIds"));
            }

        }
        catch (AggregateException exception)
        {
            Logging.WriteError(exception);
            var ex = (GraphQLHttpException) exception.InnerExceptions[0];
            Console.WriteLine(ex.HttpResponseMessage.Content.ReadAsStringAsync());
            throw;
        }

Is it possible to read the response body somehow?

[deinok]

Yes, you are right, the problem is that the Content is disposed before the object is passed to the Exception.
So, Console.WriteLine(ex.HttpResponseMessage.Content.ReadAsStringAsync()); will fail cause Content is in disposed state

[CyrilleFormont]

Any news about this issue ?

[RobinHSanner]

This issue is pretty critical because if I can't see the error that occurred then i'm totally in the dark about how to fix it. The issue could be resolved using the Changes:

[RobinHSanner]

Is this issue fixed in the latest release (3.0.1)?

[DanielStout5]

The issue is still present in 3.0.1

[brvaland]

@deinok - any update when a fix will be available or a workaround?

[rose-a]

What about using HttpResponseMessage.EnsureSuccessStatusCode to throw a HttpRequestException instead of a GraphQLHttpException? MS might have been there before...

I'll try to change this in 3.0.2...

[RobinHSanner]

Well I had the fix in my response from Mar 4 but the images are gone from the server now and I didn't keep them

[rose-a]

Yes, I still can see them, but why reinvent the wheel?

[brvaland]

@rose-a - Thanks for quick reply. Do you have ETA when 3.0.2 will be available so we can test it ?

[RobinHSanner]

It wasn't a reinvent, the current implementation was broken to such an extent that changing the implementation would have no impact on existing code because the current implementation was not usable.

[RobinHSanner]

But regardless I'd just be happy to have it resolved, however that is. Thank you