Skip to content

Conversation

@mirdono
Copy link
Member

@mirdono mirdono commented Jan 26, 2026

Warning

This PR depends on the functionality being implemented into sparql-parser. Do not merge until that PR has progressed.

Use an ODRL authorisation policy as configuration for sparql-parser. With support for ODRL configuration being added to sparql-parser we no longer need the odrl-parser-service to translate an ODRL policy to sparql-parser's lisp ACL.

  • Updates the app's database service configuration to use ODRL as implemented in sparql-parser.
  • Removes the odrl-parser service and its configuration.
  • Update the project script to convert ttl to n-triples to search in the sparql-parser configuration directory.

TODO

Before merging and deploying, also do the following:

  • Ensure ODRL policy matches most recent lisp authorisation configuration.
  • Add comment to lisp configuration warning that is it not used anymore, or remove the file from the repository.
  • Ensure a feature-branch build is available for sparql-parser, alternatively use a locally build image.

Related tickets

  • LBRON-719
  • LBRON-485

- Add write permission for harvesting graph
- Rename party collections to match lisp configuration

Note, it also important to use the exact name `public` for the "default" group,
as this is the default permission in requests.
Now that sparql-parser can use ODRL policies directly this service is not longer
needed.
@mirdono mirdono force-pushed the feat/odrl-policy-for-authz-sparql-parser branch from 1e6f2d6 to fb4b3c7 Compare January 26, 2026 13:28
@mirdono mirdono self-assigned this Jan 26, 2026
@mirdono mirdono added the enhancement New feature or request label Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants