Skip to content

Conversation

@HDauven
Copy link
Member

@HDauven HDauven commented Aug 17, 2025

Resolves #91

@HDauven HDauven requested a review from Neotamandua August 17, 2025 11:47
Neotamandua
Neotamandua previously approved these changes Aug 17, 2025
Copy link
Member

@Neotamandua Neotamandua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Added two small suggestions

@HDauven HDauven requested a review from Neotamandua August 17, 2025 12:41
@cyborgjox
Copy link

Pros Can unstake and claim rewards while using web wallet. If consensus key / public profile 1 is set to stake owner

Cons Cannot unstake or claim rewards while using web wallet, as long as profile 2 public address is set as stake owner.

(Second profile owner can only be added through rusk-wallet. The feature not available on web wallet)

For the record, I don't see the second option as a con, but a new safety feature, if someone gains access to your web wallet.😅

@HDauven
Copy link
Member Author

HDauven commented Aug 17, 2025

Pros Can unstake and claim rewards while using web wallet. If consensus key / public profile 1 is set to stake owner

Cons Cannot unstake or claim rewards while using web wallet, as long as profile 2 public address is set as stake owner.

(Second profile owner can only be added through rusk-wallet. The feature not available on web wallet)

For the record, I don't see the second option as a con, but a new safety feature, if someone gains access to your web wallet.😅

Thanks for the feedback! I'm going to leave this out as we do not promote the web wallet for this use at the moment. It still lacks features we have on our backlog for this.

@HDauven HDauven merged commit 2aec625 into main Aug 17, 2025
1 check passed
@HDauven HDauven deleted the 91/node-installer-owner-key branch August 17, 2025 17:35
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.

Content: Missing Owner-Consensus key separation in Node Installer

4 participants