Add Exception information to failed requests #375
StefanCuypers
started this conversation in
Ideas
Replies: 1 comment
-
That's a breaking change, so I wouldn't be able to accept it until the 10.x update. However, I like the idea - why don't you file an issue on it and I'll see what I can do. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I was using the Datasync library and got errors during synch. They were returned in the failed requests of the PullResult, but the status showed 200 and 'OK'. It wasn't clear at all what caused the error. I finished by pulling the source code and debugging and found out that the problem was a Guid field that should have been a Guid? field since it could be null. It would be great if that would be visible in the failed requests.
So in PullOpertationsManager on line 140: result.AddFailedRequest(requestUri, ex.ServiceResponse); -> change to also register the exception itself and not just the ServiceResponse.
Beta Was this translation helpful? Give feedback.
All reactions