jsteelepfpt commented on code in PR #290:
URL: https://github.com/apache/directory-scimple/pull/290#discussion_r1175558371
##########
scim-core/src/main/java/org/apache/directory/scim/core/json/ObjectMapperFactory.java:
##########
@@ -61,6 +61,7 @@ private static ObjectMapper createObjectMapper() {
objectMapper.setAnnotationIntrospector(pair);
objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES,
false);
+ objectMapper.configure(MapperFeature.ACCEPT_CASE_INSENSITIVE_ENUMS, true);
Review Comment:
@bdemers @erant10 This was Eran's suggested fix when I asked him about
handling "Add" vs "add" for PATCH operation values. Should probably also add a
test for this somewhere, but I just wanted to get this on people's radar
screen. Right now, we're getting failures during Azure AD SCIM syncs due to
"Add" not being accepted as a valid PATCH operation value.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
