Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stale read of smids - on multisig rotate neither GroupHab.smids nor HabitatRecord.smids are updated to latest signing members #856

Open
kentbull opened this issue Sep 4, 2024 · 1 comment · Fixed by #858
Labels
bug Something isn't working triage

Comments

@kentbull
Copy link
Contributor

kentbull commented Sep 4, 2024

Version

1.1.18

Environment

MacOS Sonoma 14.6.1, Python 3.12.3, local deployment

Expected behavior

Calling GroupHab.smids should return the latest smids sent in to a multisig rotate command.

Actual behavior

Currently the smids coming back is exactly what was sent in during the inception event, or for a member being rotated in, the smids as of them being rotated into the group.

Steps to reproduce

Perform a rotation on any multisig AID where you either rotate out one of the original group or rotate a new member in.

If you rotate a member out then both the remaining members will still have the old smids values in their HabitatRecord.smids value.

If you rotate a member in then the original members will not have the new member's AID in their HabitatRecord.smids value, which is what GroupHab.smids is populated from during the Habery.loadHabs() call. The new member will have the correct GroupHab.smids value because it was provided the correct value in the rotation event that brought the new member in, yet on the next rotation if there are any members removed or added then that smids value will then be out of date.

@kentbull kentbull added bug Something isn't working triage labels Sep 4, 2024
@kentbull
Copy link
Contributor Author

kentbull commented Sep 5, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant