-
Notifications
You must be signed in to change notification settings - Fork 44
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
Avoid BlankNodes inside PATCHes #390
base: main
Are you sure you want to change the base?
Conversation
The spec https://solidproject.org/TR/protocol#server-patch-n3-blank-nodes BlankNodes inside N3 PATCH entries, so let's use a NamedNode here? Hopefully this can fix solid-contrib/pivot#31
Ah, and the unit tests are already failing :) |
Unit tests fixed. I'll see if I can test it manually in a development setup too. |
I'm wondering if other panes have the same issue like contacts ... Isn't this is basically a rdflib issue |
I searched for occurrences of Apart from the unit tests passing, thanks to @bourgeoa I was able to test this on http://solidcommunity.net:8443 and update my trusted applications successfully. The request it makes now contains a named node instead of a blank node, and CSS accepts it:
|
I opened a question to CSS CommunitySolidServer/CommunitySolidServer#1968 |
The spec prohibits BlankNodes inside N3 PATCH entries, so let's use a NamedNode here?
Hopefully this can fix solid-contrib/pivot#31
I haven't tested this code yet.