Skip to content

Comments

Introducing conflict resolution strategies in the UMA server#78

Open
woutslabbinck wants to merge 3 commits intomainfrom
feat/defaultstrategy
Open

Introducing conflict resolution strategies in the UMA server#78
woutslabbinck wants to merge 3 commits intomainfrom
feat/defaultstrategy

Conversation

@woutslabbinck
Copy link
Contributor

Introduces a Strategy component in the ODRLAuthorizer such that the strategy is less hardcoded.

@woutslabbinck
Copy link
Contributor Author

There is a small issue which is related to componentsjs and the package policy-conflict-resolver.
As a result, building the packages results into an error:

[@solidlab/uma]: Resource https://linkedsoftwaredependencies.org/bundles/npm/policy-conflict-resolver/^0.0.0/dist/EvaluatorHandler.jsonld#EvaluatorHandler is not a valid component, either it is not defined, has no type, or is incorrectly referenced by https://linkedsoftwaredependencies.org/bundles/npm/policy-conflict-resolver/^0.0.0/dist/WrappedEvaluatorHandler.jsonld#WrappedEvaluatorHandler.
[@solidlab/uma]: Process exited (exit code 1), completed in 6s 275ms

All tests do work however if you remove all aspects of components.js in node_modules (which is dirty, but I wanted to check if I had to change more tests) from the policy-conflict-resolver.

There are two more interesting aspects that could be introduced in this PR:

  • make strategies truly configurable via componentsjs (not much work)
  • ODRL evaluator 0.6.0 introduced eyeling. We could make eyeling the default N3 Reasoner (though maybe that is better for a different branch + PR)

@joachimvh
Copy link
Contributor

There is a small issue which is related to componentsjs and the package policy-conflict-resolver.

I see that the conflict resolver library does not export the EvaluatorHandler from the index.ts, which is probably what is causing this issue.

@joachimvh
Copy link
Contributor

I have released v0.0.3 of the conflict library which should fix this

@joachimvh joachimvh force-pushed the feat/defaultstrategy branch from efa0cf6 to e13c159 Compare February 16, 2026 11:48
@joachimvh joachimvh force-pushed the feat/defaultstrategy branch from e13c159 to 96d1286 Compare February 20, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants