SimpleIdServer icon indicating copy to clipboard operation
SimpleIdServer copied to clipboard

[SCIM] PATCH replace operation remove others properties

Open gabrielemilan opened this issue 1 year ago • 7 comments

Hello, it seems there is a regression in the replace operation.

image

Some arrays have been deleted.

Can you please check? Regards.

gabrielemilan avatar Mar 14 '24 14:03 gabrielemilan

Hello,

Due to performance issues, we have made some modifications to the SimpleIdServer.Scim package. Now, it does not return the entire object or representation in the response to an HTTP PATCH or PUT request.

There are scenarios where a user can be assigned to more than one million groups. In such situations, we do not wish to return the full representation.

Could you please check if the representation is correct when you execute an HTTP GET?

If you prefer, we can add a new property, IsFullRepresentationReturned, in the options. This property would enable returning the full representation in the response when set to true.

Best regards,

SID

simpleidserver avatar Mar 14 '24 16:03 simpleidserver

Yes please, it can be a good solution.

Thanks

gabrielemilan avatar Mar 14 '24 23:03 gabrielemilan

The 'IsFullRepresentationReturned' property has been added to the options. When this property is set to true, the full representation will be returned.

simpleidserver avatar Mar 15 '24 13:03 simpleidserver

The 'IsFullRepresentationReturned' property has been added to the options. When this property is set to true, the full representation will be returned.

Ok thanks, is it the version already published?

gabrielemilan avatar Mar 15 '24 13:03 gabrielemilan

The version has not yet been published. I will release a pre-release version next week.

simpleidserver avatar Mar 16 '24 12:03 simpleidserver

The NuGet package 4.0.8-rc1 has been published. You can set the property IsFullRepresentationReturned to true in order to return the complete representation after the update operation.

simpleidserver avatar Mar 20 '24 14:03 simpleidserver

The NuGet package 4.0.8-rc1 has been published. You can set the property IsFullRepresentationReturned to true in order to return the complete representation after the update operation.

thank you very much

gabrielemilan avatar Mar 20 '24 14:03 gabrielemilan