Skip to content

Conversation

@hanzel98
Copy link
Contributor

What?

  • Implemented a rehashing of the delegationHash inside the enforcers in the LogicalOrWrapper enforcer.

Why?

  • Previously the LogicalOr enforcer allowed to have duplicate enforcers inside that shared the delegationHash as a key to control the state changes. This generated a collision in the state when using these enforcers inside the LogicalOr.

How?

  • The idea is to prevent state collisions when the delegation hash is the only key, so this PR hashes the delegationHash of the LogicalOr delegation together with the selected caveat group index. Inside the logicalOr there are groups and the user selects which group to use by choosing its index. This index makes each group produce a new delegationHash that is passed to the enforcers. Now it is possible to combine enforcers of the same type without those mapping collisions.

@hanzel98 hanzel98 requested a review from a team as a code owner May 20, 2025 18:28
@hanzel98 hanzel98 self-assigned this May 20, 2025
@hanzel98 hanzel98 requested a review from McOso May 20, 2025 18:28
@hanzel98 hanzel98 added the new release This PR goes to the next release label Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new release This PR goes to the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants