-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug: Remove /array/- raises TypeError #133
Comments
Thanks for the report. I think your suggestion makes a lot of sense. Would you be interested in preparing a pull request? |
Ok, I will find some time to do it. So according to this section, I think it make sense to raise a |
Agreed - makes sense |
FYI I made a PR, can you check please? |
Looks good! I’ll merge and include it in the next release. Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version: 1.32
Python version: 3.8
Description
Removing array element with dash notation raises an unexpected TypeError. Like so:
{'op': 'remove', 'path': '/vals/-'}
.I understand that the dash points to the end of array, but not the last element? Even so, if the syntax is invalid, I think it should raise a
JsonPatchException
orJsonPointerException
.Step to reproduce
Stack trace
The text was updated successfully, but these errors were encountered: