Skip to content

[#384] eliminate vestigal `UNAUTHORIZED` paths in execute layer

  • in MR !477 (merged) we introduced a centralized approach to handling and returning early from incoherent messages containing comamnds (including a sender issuing a command they are not authorized to issue). this is a better approach that is more cohesive and easy to reason about. yay!
  • it also makes unnecessary the scattered authorization checks in every method in execute that might only be issuable by someone with a certain membership status. the previous MR removed the tests for those code paths but not the code paths.
  • this MR removes the code paths (and cleans up execute a bit while we're at it. :))

Closes #384 (closed)

Edited by aguestuser

Merge request reports