Treasury spend origin

Currently treasury is configured with SpendOrigin = frame_support::traits::NeverEnsureOrigin<Balance>, meaning no origin is authorized to spend.

If I understand correctly, there is currently no way to use the treasury.

Before a democratic system is implemented, should we leave the treasury as-is, or authorize sudo or technical committee?

When this is done, attention should be given to a potentially unused imbalance in impl<Runtime> pallet_treasury::SpendFunds<Runtime> for TreasurySpendFunds<Runtime> (global search for _imbalance).

added C-runtime D-easy ET-days I-ToPrioritize P7-nicetohave RN-runtime S-needdiscussion labels

changed title from Trasury spend origin to Treasury spend origin

With the latest ParityTech fix on the benchmarks of the pallet treasury, it is now insecure to have a SpendOrigin that always fails and a RejectOrigin that can succeed. This is our current setup, so in the next SDK upgrade, this issue should be addressed, or at least RejectOrigin should be set to never succeed temporary.

Fixed with https://github.com/paritytech/polkadot-sdk/commit/9cdbdc5a73f3c2d1d9afdf266799f249c3c04c2d

removed RN-runtime label

assigned to @bgallois

mentioned in commit 830721e5

mentioned in commit 2d972d7c

mentioned in commit 19b13b75