GetEnterpriseUsersWithMarkerAsync throws NRE when evaluating Entries

[watfordsuzy]

• I have checked that the SDK documentation doesn't solve my issue.
• I have checked that the API documentation doesn't solve my issue.
• I have searched the Box Developer Forums and my issue isn't already reported (or if it has been reported, I have attached a link to it, for reference).
• I have searched Issues in this repo and my issue isn't already reported.

Description of the Issue

If I include an externalAppUserId to a call to GetEnterpriseUsersWithMarkerAsync the resulting marker based collection has a null Entries field.

Steps to Reproduce

string? marker = null;
do
{
    var result = await client.UsersManager.GetEnterpriseUsersWithMarkerAsync(marker, externalAppUserId: "user_id");
    foreach (BoxUser user in result.Entries) // NullReferenceException (result.Entries)
    {
        // ...
    }

    marker = result.NextMarker;
}
while (!string.IsNullOrWhiteSpace(marker));

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

[watfordsuzy]

Worse still, calling GetEnterpriseUsersAsync also throws an error when I include the external app ID.

[watfordsuzy]

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/

[mwwoda]

I just checked and this endpoint behaves very strangely with different payload. When useMarker is included along with external_app_user_id, instead of returning collection in entries field as usual it returns it in the users field, which is not even present in the spec. As a result SDK cannot deserialize it because of field name mismatch and entries field is null.

I could only reproduce this issue only when used marker so far (couldn't reproduce with GetEnterpriseUsersAsync only with GetEnterpriseUsersMarkerBasedAsync). At the first glance, response in your example from GetEnterpriseUsersAsync looks fine. Not sure why you got NRE here.

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.

[mwwoda]

You also mentioned regression? Could you elaborate on that? Was it working fine for you in the past?

[watfordsuzy]

Yes, all of this code was working for me previously. Seems only recently that the addition of external_app_user_id causes the weird response from the server.

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

[watfordsuzy]

It looks like you were correct and I can successfully work around the issue using GetEnterpriseUsersAsync. Not 100% sure why I saw the NRE, might have been an old instance of the code was picked up by the function app host.

[mwwoda]

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 GetEnterpriseUsersAsync function.

[watfordsuzy]

@mwwoda thank you, will do!

Is it even possible to have two box users with the same external_app_user_id?