-
Notifications
You must be signed in to change notification settings - Fork 163
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
GetEnterpriseUsersWithMarkerAsync throws NRE when evaluating Entries #948
Comments
Worse still, calling |
I now think this is a regression with the main Box API, how do I get this escalated? https://gist.github.com/watfordsuzy/c5f85f1099e259f64e5acb4b0881d18f ref: https://developer.box.com/guides/api-calls/pagination/marker-based/ |
I just checked and this endpoint behaves very strangely with different payload. When I could only reproduce this issue only when used marker so far (couldn't reproduce with You could try contacting Box support at https://forum.box.com/top?period=quarterly. I'll contact the service owners on our side to get first-hand feedback. |
You also mentioned regression? Could you elaborate on that? Was it working fine for you in the past? |
Yes, all of this code was working for me previously. Seems only recently that the addition of EDIT: I've also raised an issue on the new support forums. https://support.box.com/hc/en-us/community/posts/29468435097747-Users-API-regression-with-usemarker-parameter |
It looks like you were correct and I can successfully work around the issue using |
Service owners confirmed the issue and they looking into a fix. I don't have any timeframe for now, but I'll let you know in case of any updates. For now, my advice is to stick with |
@mwwoda thank you, will do! Is it even possible to have two box users with the same |
Hi @watfordsuzy As I tested, so it's not possible to have two Box users with the same |
@mwwoda @congminh1254 I received the following from Box Support on my case and I'm not sure how to manage this with the C# library:
I asked a follow-up question about the stability of the APIs when requesting Marker Based results and received the following:
|
This issue has been automatically marked as stale because it has not been updated in the last 30 days. It will be closed if no further activity occurs within the next 7 days. Feel free to reach out or mention Box SDK team member for further help and resources if they are needed. |
@mwwoda @congminh1254 thoughts on this? I think the SDK should be modified such that this doesn't break unexpectedly. Either by:
|
@watfordsuzy |
Description of the Issue
If I include an
externalAppUserId
to a call toGetEnterpriseUsersWithMarkerAsync
the resulting marker based collection has a nullEntries
field.Steps to Reproduce
Expected Behavior
The list of entries is enumerated.
Error Message, Including Stack Trace
This is in a durable function app and the stack trace is a bit useless.
Screenshots
Versions Used
.Net SDK: 5.7.0
Windows: dotnet-isolated|8.0
The text was updated successfully, but these errors were encountered: