-
Notifications
You must be signed in to change notification settings - Fork 546
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
Fix for issue1957 sparql parser percent encoded reserved chars #1959
Conversation
pre-commit.ci autofix |
…cent-encoded-reserved-chars
There is no need to expand percentage encoded parts of PN_LOCAL, as they should just be directly be used in the resulting IRI, and escaping them just risks creating an invalid IRI. Also: - Move tests to `test/test_sparql/test_prefixed_name.py` - Add some more general tests for prefixed names.
@gjhiggins made a commit on your branch:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, good to have this fixed.
…cent-encoded-reserved-chars
Summary of changes
Seems like
_hexExpand
internal SPARQL parser function inappropriately expands percent-encoded reserved characters. Added an exclusionary regexp to disable this behaviour and a parameterized test which checks SPARQL parser processing of the set of percent-encoded reserved charsChecklist
so maintainers can fix minor issues and keep your PR up to date.