-
Notifications
You must be signed in to change notification settings - Fork 44
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
Critical Bug in DownloadReportAsyncWithHttpInfo #36
Comments
As I remember i also found this problem and fix it in my form to make sure that i can use API and download report - this is main functionality for now that required to work. I am continue using my solution that works. @xantari try to take a look on my fork - at least downloading reports is working. Thanks. |
@Digiman Thanks! I've started fixing the pieces so far that I have found for just the reporting infrastructure. It's unbelievable to me that only one Cybersource developer is assigned to this (and only part time as indicated by his checkin requests accross Python, Ruby, PHP, and C# librarys). It's amazing to me that Cybersource is telling people to switch to the new REST API by 7/31/2019, but don't provide the proper client library abstractions to do so. Talk about the cart before the horse! Upper management at Cybersource should be rethinking their 7/31/2019 deadline in light of these issues. But alas, it is probably falling on deaf ears. |
@xantari Yep, I think so. It's really bad approach. I had discussion with guy from CyberSource (some kind manager) that talking and talking without stop. I said that library need to be improved but ... nothings changed. It's part of he problem. Also they had issues inside report. It's more fully - it like as they don't have any testing (QA). At least for now - my solution working. Hope it will work in future. Maybe in 1-2 years they will provide proper logic and working library) |
Cybersource support case # 06135500 stated this works fine for them. They said "For issue 36, our engineers were able to receive the report information back in the console." Here is the failing lines of code: cybersource-rest-client-dotnet/Api/ReportDownloadsApi.cs Lines 335 to 337 in d235961
Here is the non-async working version lines of code: cybersource-rest-client-dotnet/Api/ReportDownloadsApi.cs Lines 258 to 260 in d235961
You will notice that in the working non-async version above you have this: localVarResponse.Content But it is missing in the Async version. This is where your bug is. You are not returning the content of the report in the Async version. |
Hi, Thank you for bringing this to our attention. This issue has been addressed in the latest version of the SDK. |
Appears to be fixed |
In the DownloadReportAsyncWithHttpInfo class when requesting a report you do not get any report data back!
I decompiled your library and found the issue.
I have highlighted issue in RED:
If you look at the non async version you get this:
You are missing restResponse.get_Content()!!!!
You are requesting everyone to convert to new REST Report download by 7/31/2019, but have such critical bug to disallow async function???
The text was updated successfully, but these errors were encountered: