Skip to content
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

Specify common data shape for all LORC notifications #10

Open
csarven opened this issue Aug 31, 2017 · 1 comment
Open

Specify common data shape for all LORC notifications #10

csarven opened this issue Aug 31, 2017 · 1 comment

Comments

@csarven
Copy link
Member

csarven commented Aug 31, 2017

Provenance level data eg actors, datetime, license, are good candidates.

@csarven csarven changed the title Specify common data shape for all notifications Specify common data shape for all LORC notifications Sep 5, 2017
@csarven
Copy link
Member Author

csarven commented Sep 5, 2017

To take one step back, part of this data shape depends on what should be provided by the sender, and if at all, what should be modified by the receiver.

As a general rule of thumb, it is better to let the receiver set the datetime to prevent misinformation altogether. If LORC commits to using CC0 license, then the sender doesn't need to provide that either. In the case that both are set, the server should probably override them. This information needs to be stated in the documentation so that senders are well-aware before sending.

As for the actor (the agent that's sending it), this could be left as a SHOULD instead of a MUST. It is nice to have this information, but I can't of anything malfunctioning if it is not set. Second of all, there is no way to verify the value without authenticating the sender or having them sign the notification.

Whatever the server decides to do (modify or not) is orthogonal in any case.

Perhaps there is no need to expect a set of statements that should be in the notification. The only ones would be the generic parts like:

@prefix oa: <http://www.w3.org/ns/oa#> .
<>  a as:Announce .

Aside: <> a as:Announce is presumably required for all notifications. The remainder of the shape under <> will be case dependent. Eventually the base URL will be the notification URL.

In any case, since there is nothing significant here, better to punt the problem to specific data shapes. No need to add a layer of requirement here.

Leaving this issue open for awhile to get feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant