simpleidserver / SimpleIdServer

OpenID, OAuth 2.0, SCIM2.0, UMA2.0, FAPI, CIBA & OPENBANKING Framework for ASP.NET Core
https://simpleidserver.com/
Apache License 2.0
719 stars 94 forks source link

[SCIM] Exception filtering by userName #565

Closed gabrielemilan closed 1 year ago

gabrielemilan commented 1 year ago

Hello guys,

I face this error in the last version:

image

calling

/v2/Users?filter=userName eq "testuser"

I tried to check the source code to fund the issue, but I don't understand why.

Regards. Gabriele

gabrielemilan commented 1 year ago

Maybe found the issue, is a regression on this commet

https://github.com/simpleidserver/SimpleIdServer/commit/babdac33e366b34ee4ceb8b5f328f40d52cdc3d2

the row:

e1 = Expression.Coalesce(e1, Expression.Constant(string.Empty));

simpleidserver commented 1 year ago

Hello,

The LINQ Expression {ValueString} ?? "" is not supported by the Nuget package MongoDB.Driver version 2.11.6 but by the latest one, and it causes this exception. I made some modifications in the branch release/v4.0.2 to fix this issue. A release is planned this week with all the changes : https://github.com/users/simpleidserver/projects/1

KR,

SID

gabrielemilan commented 1 year ago

Tested on 4.0.3