wso2-extensions / identity-inbound-provisioning-scim2

Apache License 2.0
8 stars 152 forks source link

Fix multi-attribute user filter with count provided. #555

Closed Thisara-Welmilla closed 4 months ago

Thisara-Welmilla commented 5 months ago

Issue: https://github.com/wso2/product-is/issues/20496

When count(limit) > 0, use the getMultiAttributeFilteredUsersWithMaxLimit with limit is provided as the given count given.

Adiition to that this contain the fix to honor the provided count value in the SCIM2 request for multi attribute filtering flow. Ported the logic(used in the single attribute filtering flow) with minor modification: https://github.com/wso2-extensions/identity-inbound-provisioning-scim2/blob/master/components/org.wso2.carbon.identity.scim2.common/src/main/java/org/wso2/carbon/identity/scim2/common/impl/SCIMUserManager.java#L1885-L1914

CLAassistant commented 5 months ago

CLA assistant check
All committers have signed the CLA.

jenkins-is-staging commented 4 months ago

PR builder started Link: https://github.com/wso2/product-is/actions/runs/9843148323

jenkins-is-staging commented 4 months ago

PR builder completed Link: https://github.com/wso2/product-is/actions/runs/9843148323 Status: failure

jenkins-is-staging commented 4 months ago

PR builder started Link: https://github.com/wso2/product-is/actions/runs/9855076719

jenkins-is-staging commented 4 months ago

PR builder completed Link: https://github.com/wso2/product-is/actions/runs/9855076719 Status: failure

jenkins-is-staging commented 4 months ago

PR builder started Link: https://github.com/wso2/product-is/actions/runs/9974859708

jenkins-is-staging commented 4 months ago

PR builder completed Link: https://github.com/wso2/product-is/actions/runs/9974859708 Status: success