Patch.Validate() does not support patch operation that is not lowercase

imulab / go-scim

Building blocks for servers implementing Simple Cloud Identity Management v2

MIT License

145 stars 56 forks source link

Patch.Validate() does not support patch operation that is not lowercase #54

Closed alexvalenzuela closed 4 years ago

alexvalenzuela commented 4 years ago

Hi, after checking the method and compare to the following commit, it seems that we also need to add this line, so both for range for PATCH operations be in compliance.

The reason is that Azure Active Directory PATCH op is capitalized. Why do they do this since this is not SCIM compliant? No one knows.

See here for more details.

imulab commented 4 years ago

Got it, will fix it with the next release.

imulab commented 4 years ago

@alexvalenzuelac issue is fixed in pkg/v2.1.1, thanks!