uncaught exception on a thread causing APPCRASH about quickbooks-v3-dotnet-sdk HOT 8 CLOSED

Can you please confirm the steps to replicate this exception? Which version of SDK are you using?

from quickbooks-v3-dotnet-sdk.

IppDotNetSdkForQuickBooksApiV3 version .2.9.1, but none of the code has changed since that version. The issue is that the BatchAsyncompleted method only catches WebExceptions - all other exceptions are left uncaught on an async thread and thus crash the process.

The CDC api was being queried.

from quickbooks-v3-dotnet-sdk.

Can you update to the latest .Net SDK version 3.1.0 and share the raw request and response from the SDK so that I can replicate the issue. On another note, we do not support CDC call in Batch. Only CRUD and Query is supported in Batch.

from quickbooks-v3-dotnet-sdk.

Additionally I check that the Exception Handling for Batch is same as for regular(non Batch) calls. So the issue really might be with usage of CDC in batch.

from quickbooks-v3-dotnet-sdk.

@ribbles, sorry for a stupid question, but have you subscribed to batch.OnBatchExecuteAsyncCompleted event before calling batch.ExecuteAsync()? I see that there is no null check in code so it will throw NullReferenceException in such case:

     batchCompletedEventArgs.Error = eventArgs.Error;
     this.OnBatchExecuteAsyncCompleted(this, batchCompletedEventArgs);

from quickbooks-v3-dotnet-sdk.

@nimisha84 we are still seeing the issue intermittently. We will try the fix by @timurka-xalien so we can see what the underlying Exception is, and this should take care of the APPCRASH. Why did you close the issue?

from quickbooks-v3-dotnet-sdk.

@ribbles - I did not see any updates from you on this for long. Also, as mentioned earlier CDC is a not supported operation for Batch. Therefore the issue was close. I have reopened it. Let me know what additional questions you have.

from quickbooks-v3-dotnet-sdk.

@nimisha84 the fix provided by @timurka-xalien worked for us. Would you accept his fix so that we can move back to your code base?

from quickbooks-v3-dotnet-sdk.