-
Notifications
You must be signed in to change notification settings - Fork 241
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
fix_: mitigate permission stuck in pending state #5070
fix_: mitigate permission stuck in pending state #5070
Conversation
Jenkins BuildsClick to see older builds (83)
|
67327a6
to
cd10710
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To reproduce the issue I would base on TestControlNodeUpdateSigner
. I'll try to come up with a test scaffold for you today.
6fe326b
to
212eebc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, other then some minor quality improvements, looks good.
@mprakhov can you please have a look?
Hi there, |
212eebc
to
04dd0f8
Compare
d9170f0
to
3dde1e8
Compare
3dde1e8
to
295db89
Compare
7e72ab1
to
51cb49e
Compare
This PR mitigates permission stuck in pending state upon making device a control node. It fixes [#14023](status-im/status-desktop#14023)
51cb49e
to
7d8f819
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look really solid!
@@ -71,6 +71,18 @@ const ( | |||
maxChunkSizeBytes = 1500000 | |||
) | |||
|
|||
const ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kudos on keeping all the errors in one place!
Description
This PR mitigates permission stuck in pending state upon making device a control node. It fixes #14023 by applying events and publishing community events once the device is promoted to control node.
System Design
The main design decision for this issue is that promoteSelfToControlNode is inhibited when the community doesn't have a minted Owner token. From the UI perspective, the action is disabled similar to when the user is an
admin
user. And from the backend perspective, the action is also rejected with missing owner token code.