Throw BadRequestException instead of server error for SCIM2 users & groups filtering with invalid filter operation #531
Conversation
…roups filtering with invalid filter operation
|
PR builder started |
| @@ -1405,7 +1405,7 @@ private UsersGetResponse filterUsersBySingleAttribute(ExpressionNode node, Map<S | |||
| if (isFilteringNotSupported(node.getOperation())) { | |||
| String errorMessage = | |||
| "Filter operation: " + node.getOperation() + " is not supported for filtering in users endpoint."; | |||
| throw new CharonException(errorMessage); | |||
| throw new BadRequestException(errorMessage); | |||
There was a problem hiding this comment.
Here you can pass the scimType variable as well
As per the attached screenshot, the value in scimType should go to details
scimType should be one of https://datatracker.ietf.org/doc/html/rfc7644#:~:text=A%20SCIM%20detail%20error%20keyword.%20%20See%20Table%209.%20%20OPTIONAL.
There was a problem hiding this comment.
Hope this is the expected result improved by this commit - cc79a63
|
PR builder completed |
There was a problem hiding this comment.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/7853504567
$subject
Related Issues