0x5bfa / FluentHub

The best, stylish yet powerful GitHub client for Windows.
MIT License
1.06k stars 51 forks source link

Bug: ThrowInt32OverflowException happens in dashboard #470

Open dran1x opened 3 months ago

dran1x commented 3 months ago

Before you start

What version of Windows is this issue present?

Windows 11

What version/architecture of FluentHub are you on?

0.8.0.60 X64 RELEASE

Description

I installed the application through the Microsoft Store, and once I ran it and signed in with my GitHub account, it started pulling my repositories, then an error showed up on the screen.

Steps To Reproduce

  1. Install application.
  2. Open and sign in.
  3. Once it starts pulling your repositories it should error.

Expected behavior

Pull all repositories and then show dashboard.

Stack Trace

Stack trace details
``` at System.Convert.ThrowInt32OverflowException() at System.Convert.ToInt32(Int64 value) at System.Convert.ChangeType(Object value, Type conversionType, IFormatProvider provider) at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1437 at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1492 at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1492 at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1492 at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1519 at Octokit.SimpleJson.DeserializeObject[T](String json, IJsonSerializerStrategy jsonSerializerStrategy) in /_/Octokit/SimpleJson.cs:line 596 at Octokit.Internal.JsonHttpPipeline.DeserializeResponse[T](IResponse response) in /_/Octokit/Http/JsonHttpPipeline.cs:line 62 at Octokit.Connection.Run[T](IRequest request, CancellationToken cancellationToken, Func`2 preprocessResponseBody) in /_/Octokit/Http/Connection.cs:line 696 at Octokit.ApiConnection.GetPage[TU](Uri uri, IDictionary`2 parameters, String accepts, ApiOptions options, Func`2 preprocessResponseBody) in /_/Octokit/Http/ApiConnection.cs:line 672 at Octokit.ApiConnection.<>c__DisplayClass19_0`1.<b__0>d.MoveNext() in /_/Octokit/Http/ApiConnection.cs:line 230 --- End of stack trace from previous location --- at Octokit.ApiPagination.GetAllPages[T](Func`1 getFirstPage, Uri uri) in /_/Octokit/Clients/ApiPagination.cs:line 23 at FluentHub.Octokit.Queries.Users.ActivityQueries.GetAllAsync(String login) at FluentHub.App.ViewModels.Viewers.DashBoardViewModel.LoadHomeContentsAsync() at FluentHub.App.ViewModels.Viewers.DashBoardViewModel.LoadUserHomePageAsync() ```

Assets

image Log20240406.log

just-a-hriday commented 1 month ago

Happening to me as well. Seems like a nice app but I can never get past this screen.

0x5bfa commented 1 month ago

Thank you for the feedback! This looks like sever issue.

Could you check:

beecho01 commented 1 month ago

I to have this issue. To answer the above questions:

This issue appears to be solely related to the "Dashboard" page on application load and appears when repositories have appeared to fill on the sidebar.


System.OverflowException: Value was either too large or too small for an Int32.
   at System.Convert.ThrowInt32OverflowException()
   at System.Convert.ToInt32(Int64 value)
   at System.Convert.ChangeType(Object value, Type conversionType, IFormatProvider provider)
   at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1437
   at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1492
   at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1492
   at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1492
   at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in /_/Octokit/SimpleJson.cs:line 1519
   at Octokit.SimpleJson.DeserializeObject[T](String json, IJsonSerializerStrategy jsonSerializerStrategy) in /_/Octokit/SimpleJson.cs:line 596
   at Octokit.Internal.JsonHttpPipeline.DeserializeResponse[T](IResponse response) in /_/Octokit/Http/JsonHttpPipeline.cs:line 62
   at Octokit.Connection.Run[T](IRequest request, CancellationToken cancellationToken, Func`2 preprocessResponseBody) in /_/Octokit/Http/Connection.cs:line 696
   at Octokit.ApiConnection.GetPage[TU](Uri uri, IDictionary`2 parameters, String accepts, ApiOptions options, Func`2 preprocessResponseBody) in /_/Octokit/Http/ApiConnection.cs:line 672
   at Octokit.ApiConnection.<>c__DisplayClass19_0`1.<<GetAll>b__0>d.MoveNext() in /_/Octokit/Http/ApiConnection.cs:line 230
--- End of stack trace from previous location ---
   at Octokit.ApiPagination.GetAllPages[T](Func`1 getFirstPage, Uri uri) in /_/Octokit/Clients/ApiPagination.cs:line 23
   at FluentHub.Octokit.Queries.Users.ActivityQueries.GetAllAsync(String login)
   at FluentHub.App.ViewModels.Viewers.DashBoardViewModel.LoadHomeContentsAsync()
   at FluentHub.App.ViewModels.Viewers.DashBoardViewModel.LoadUserHomePageAsync()
0x5bfa commented 1 month ago

Thank you for the detailed information. I confirmed this has been fixed in the latest API package.

I'll confirm if really fixed by seeing those pages you listed 🙏

beecho01 commented 1 month ago

Thank you for the quick reply. Look forward to the new release.