-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Retrieving events throws Stack.Overflow exception. #1979
Comments
It could be another field that needs to change from Are you able to post a code snippet that reproduces it on a public repository? |
1 similar comment
It could be another field that needs to change from Are you able to post a code snippet that reproduces it on a public repository? |
I was just creating a github client in powershell and trying to query all events for a repo. As for the code, I didnt change it. I built directly from the cloned repo. I'll admit maybe I did something wrong, but I really can't guess what. |
@bobbytreed if you can reproduce this against a public repository and can share that with us it'll help us to reproduce the issue on our end and identify what field returned from the API now overflows |
I fixed it locally by changing IssueEvent ID to a long as well. I still hit an issue though with pagination. Exception : System.AggregateException: One or more errors occurred. ---> Octokit.ApiValidationException: In order to |
@bobbytreed some sample code would still be handy so we can understand how you're encountering issues |
$inmem = new-object OctoKit.Internal.InMemoryCredentialStore -ArgumentList ($cred) |
@bobbytreed apologies for the delay. I see that this last line doesn't have the owner and repo values:
If this is reproducible on a public repository that'll greatly help to speed up reproducing this issue. |
This |
This fails for me with the following exception:
NOTE: I guess mine is a different issue... |
Any update on this? The bug exists on latest version (9.1.2) Inner Exception (System.OverflowException): Value was either too large or too small for an Int32. |
Same issue here: #2889 |
Facing the same issue all of a sudden, any solutions? "Value was either too large or too small for an Int32."
|
Update top 10.0. It's solved. |
In the lastest local build (0.33.0), I still get a StackOverflow Exception. While the type has been updated, something in PocoSerializationStrategy is still trying to convert to Int32
System.AggregateException: One or more errors occurred. ---> System.OverflowException: Value was either too large or too small for an Int32.
at System.Convert.ToInt32(Int64 value)
at System.Convert.ChangeType(Object value, Type conversionType, IFormatProvider provider)
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type)
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type)
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type)
at Octokit.SimpleJson.DeserializeObject[T](String json, IJsonSerializerStrategy jsonSerializerStrategy)
at Octokit.Internal.JsonHttpPipeline.DeserializeResponse[T](IResponse response)
at Octokit.Connection.d__57
1.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Octokit.ApiConnection.<GetPage>d__42
1.MoveNext()--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.ApiConnection.<>c__DisplayClass17_0
1.<<GetAll>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Octokit.ApiPagination.<GetAllPages>d__0
1.MoveNext()--- End of inner exception stack trace ---
---> (Inner Exception #0) System.OverflowException: Value was either too large or too small for an Int32.
at System.Convert.ToInt32(Int64 value)
at System.Convert.ChangeType(Object value, Type conversionType, IFormatProvider provider)
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type)
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type)
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type)
at Octokit.SimpleJson.DeserializeObject[T](String json, IJsonSerializerStrategy jsonSerializerStrategy)
at Octokit.Internal.JsonHttpPipeline.DeserializeResponse[T](IResponse response)
at Octokit.Connection.d__57
1.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Octokit.ApiConnection.<GetPage>d__42
1.MoveNext()--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.ApiConnection.<>c__DisplayClass17_0
1.<<GetAll>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Octokit.ApiPagination.<GetAllPages>d__0
1.MoveNext()<---The text was updated successfully, but these errors were encountered: