Skip to content

fix(did): revoke issuers on deactivation#1035

Closed
modelsbridgeaicom-ship-it wants to merge 1 commit into
openmetaearth:mainfrom
modelsbridgeaicom-ship-it:codex/did-deactivate-revoke-issuers-267
Closed

fix(did): revoke issuers on deactivation#1035
modelsbridgeaicom-ship-it wants to merge 1 commit into
openmetaearth:mainfrom
modelsbridgeaicom-ship-it:codex/did-deactivate-revoke-issuers-267

Conversation

@modelsbridgeaicom-ship-it

Copy link
Copy Markdown
Contributor

Summary

  • remove a DID from every service issuer list when UpdateDidStatus deactivates it
  • keep reactivation from silently restoring prior issuer memberships without explicit re-authorization
  • remove duplicate issuer entries as part of the revocation pass
  • add a message-server regression test with a fake GlobalDao keeper

Fixes #267.

Tests

  • ..\..\tools\go\bin\go.exe test .\x\did\keeper -run TestUpdateDidStatusInactiveRevokesServiceIssuerMemberships -count=1 -v
  • ..\..\tools\go\bin\go.exe test .\x\did\keeper -count=1
  • git diff --check
  • git diff --cached --check

..\..\tools\go\bin\go.exe test .\x\did -count=1 is currently blocked by the existing x/did/genesis_test.go fixture, which panics in InitGenesis with empty address string is not allowed.

@koelzen koelzen added run-ci Trigger CI run and removed run-ci Trigger CI run labels Jun 10, 2026
@marvin-985 marvin-985 closed this Jun 24, 2026
@marvin-985 marvin-985 reopened this Jun 24, 2026
@marvin-985 marvin-985 closed this Jun 24, 2026
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.

[BUG BOUNTY] [High] [DID] DID Deactivation Does Not Revoke Service Issuer Membership -- Reactivation Silently Restores All Privileges

3 participants