Skip to content

Commit

Permalink
Report "latest set value" in ACL change event
Browse files Browse the repository at this point in the history
  • Loading branch information
lauckhart committed Nov 24, 2024
1 parent 7d4e240 commit f92f587
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,9 @@ export class AccessControlServer extends AccessControlBehavior {
latestValue:
(changeType === AccessControlTypes.ChangeType.Removed
? oldFabricExtensions[0]
: fabricExtensions[0]) ?? null,
: // Unclear why this field is nullable... It does seem like the latest value is "none". But
// certification tests fail unless we report "latest non-null value"
fabricExtensions[0]) ?? oldFabricExtensions[0],
fabricIndex: relevantFabricIndex,
},
this.context,
Expand Down

0 comments on commit f92f587

Please sign in to comment.