Stack overflow exception when using client.Query<T> about sahb.graphqlclient HOT 7 CLOSED

Hi,

I unfortunately cannot replicate the issue. Can you send me a proof of concept code?

This is the full sample code from the first example for the readme.

        static async Task Main(string[] args)
        {
            IGraphQLHttpClient client = GraphQLHttpClient.Default();

            // Get response from url
            var response = await client.Query<Query>("https://mpjk0plp9.lp.gql.zone/graphql");

            // Get name etc.
            Console.WriteLine(response.Hero.Name);
            Console.ReadKey();
        }

        // The query class used is
        public class Query
        {
            public CharacterOrPerson Hero { get; set; }
        }

        public class CharacterOrPerson
        {
            public string Name { get; set; }
            public IEnumerable<Friend> Friends { get; set; }
        }

        public class Friend
        {
            public string Name { get; set; }
        }

Output:
R2-D2

The csproj is:

<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.0</TargetFramework>
    <LangVersion>7.1</LangVersion>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="SAHB.GraphQLClient" Version="1.0.1" />
  </ItemGroup>

</Project>

from sahb.graphqlclient.

Aha, using your settings (i.e. .net core 2), I don't have the issue either.
Unfortunately, I'm bound to use .Net Framework 4.6.1 for my other project right now and with this framework version, the issue persists.

Do you plan to add support for .Net Framework? Or which versions are supported currently?

Best regards
Martin

from sahb.graphqlclient.

Hello Martin

The library is intended to support .NET Framework 4.5.2. I will try to adjust the unit tests so they also run for .NET Framework 4.5.2.

I will try to make a update which should fix it. Stay tuned!

from sahb.graphqlclient.

Please try 1.0.2-beta0001 on Nuget and tell me if it works.

from sahb.graphqlclient.

I can confirm, that the new build works with .Net 4.6.1 and 4.7 as well 👍

from sahb.graphqlclient.

I will then publish 1.0.2 final later today. Thanks for your bug report :)

from sahb.graphqlclient.

1.0.2 published on NuGet

from sahb.graphqlclient.