-
Notifications
You must be signed in to change notification settings - Fork 7
JSON-LD structural requirements have issues #105
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
Comments
I agree with this in general. IMO the consumer should be responsible for providing While in very simple cases, where all the statements have the same IRI as subject or object (and leverage JSON-LD
I think we should aim for that. I think we can still mandate |
I was thinking of just following LDP and Solid Protocol support for both As I understand Solid storage doesn't guarantee that a specific JSON-LD structure gets preserved. For example in a quad store-backed CSS, creating a resource with a specific JSON-LD payload wouldn't guarantee the same structure when requesting that resource with I'm afraid that if we start relying on specific context and frame, solid storage will be unfit for storing that data. I will probably raise this issue again for Solid-OIDC, since it seems that apps can't publish their Client ID Documents to solid storage in a reliable way. |
There has been some updates to the spec that's formerly known as the "Notification Subscription Metadata Resource". It is currently known as the "Notification Channel Resource". The section on discovery clarifies the requirements: https://solid.github.io/notifications/protocol#notification-channel-discovery but the concerns on JSON-LD as per this issue still stands. I agree that the spec needs a revisit pertaining to JSON-LD. I do not think we need to revisit serializations in general. I'd however be in favour of making sure things are uniform as much as possible for the notification channel resource, subscription request and response, and notification. The following needs to be revisited:
I'm on board with some of the possibilities mentioned in earlier comments. Here is my take/suggestion: The JSON-LD compacted document form ( https://www.w3.org/TR/json-ld11/#compacted-document-form , https://www.w3.org/TR/json-ld11/#application-ld-json ) should be sufficient to keep things RDF based and friendly-enough (for plain JSON consumers, if they so wish). Essentially We may also want to describe |
Thinking more on this. I think one of our goals for this spec should be to remain compatible with the Solid Protocol. So, if the Notification Channel Resource, Subscription Resource, and Notifications (for some Subscription Types) is on a Solid server (and I think we somewhat expect that but the spec may not explicitly mandate that), the server shouldn't have to do additional work. Regarding a specific JSON-LD serialization, I suppose while a Solid server may accept a particular JSON-LD document form (like compacted) but it may never actually use the same form in a response payload. In that case, if the Notifications Protocol requires compacted JSON-LD for example, we are asking the Solid server to do a bit more. I think it may be preferable to not ask that additional work. It should work out of the box. And so the Notifications Protocol should only say that JSON-LD is required and leave it at that. The Notifications Protocol, or at least the core Notification Data Model must be RDF-based - I don't expect anyone objecting that, I hope - and if anything, be JSON-LD friendly. There are a couple of specs that make use of JSON-LD that can work with the Notifications Protocol. Minimal interop is already in place in some cases. I worry that if we say something other than JSON-LD, we may be complicating things for both servers and clients. (I say this without being a fan of JSON-LD... hey, ask me what my fave is :) but trying to make the best of aligning specs.) |
I re-raised Solid-OIDC Client ID Document requirements in solid/solid-oidc#95 |
Considerations/decisions elsewhere that can be useful input: |
A proposal to resolve this issue: #122 that focuses on making the requirements uniform. |
No description provided.
The text was updated successfully, but these errors were encountered: