Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

allow root to use sudo #9932

Closed
wants to merge 1 commit into from
Closed

allow root to use sudo #9932

wants to merge 1 commit into from

Conversation

xlc
Copy link
Contributor

@xlc xlc commented Oct 5, 2021

Allow root to use set_key and sudo_as.

Useful to replace force_transfer etc. No need to implement force_xxx for all different kinds of operations.

This will make sudo_as useable for democracy.

@shawntabrizi
Copy link
Member

shawntabrizi commented Oct 5, 2021

Sounds like the change you actually want to make is to create a call_as root function in the System pallet.

I don't see this PR as qualifying to remove force_transfer in balances, as we do not assume most runtimes have the Sudo pallet, so this kind of functionality would be useful for those chains.

If you added this to System, then maybe we could make such claims.

I would also say this PR is very strange. The whole point of the Sudo pallet is to gain access to the Root origin, but if you already have access for that check to pass, then why funnel your call through Sudo at all?

@xlc
Copy link
Contributor Author

xlc commented Oct 5, 2021

ok. I will add a call_as to system pallet.

The reason I want this is because there is not other way to call method on behalf of another signed origin. Otherwise I will need to implement different kinds of force_xxx to many our pallets.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants