search

KevinDockx / JsonPatch

JSON Patch (JsonPatchDocument) RFC 6902 implementation for .NET
MIT License
174 stars 28 forks source link

Improve error message #35

Closed KevinDockx closed 8 years ago

KevinDockx commented 9 years ago

=> get 'm in line with those from the Asp.NET version

dgioulakis commented 9 years ago

Would love to see this. Specifically, JsonPatch error codes as constants. I'm using this library to implement the SCIM specification (https://github.com/PowerDMS/Owin.Scim).

However, SCIM's error response handling is very specific concerning patch operations. See: SCIM Error Handling - invalidFilter, invalidPath, noTarget, etc

Having explicit error codes to represent reasons for a JsonPatchException to occur would be quite helpful. However, the SCIM patch is not an exact implementation of this RFC so I'll have to use a custom ObjectAdapter.

Thanks for your hard work!

KevinDockx commented 9 years ago

That's a good suggestion, tnx :) I'll look into it!

KevinDockx commented 8 years ago

Currently the error messages are on par with the other versions of this library. Going to close this for now.