2023-01-20 11:00:18 -05:00
|
|
|
#![cfg_attr(docsrs, feature(doc_cfg))]
|
|
|
|
|
#![cfg_attr(docsrs, feature(doc_auto_cfg))]
|
|
|
|
|
#![cfg_attr(not(feature = "std"), no_std)]
|
|
|
|
|
|
2023-03-26 08:43:01 -04:00
|
|
|
use scale::Encode;
|
2023-01-20 11:00:18 -05:00
|
|
|
|
2023-09-29 03:51:01 -04:00
|
|
|
use sp_io::hashing::blake2_256;
|
2023-01-20 11:00:18 -05:00
|
|
|
use sp_runtime::RuntimeDebug;
|
|
|
|
|
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
use serai_primitives::{BlockHash, NetworkId};
|
2023-01-20 11:00:18 -05:00
|
|
|
|
|
|
|
|
pub use in_instructions_primitives as primitives;
|
2023-05-13 04:20:13 -04:00
|
|
|
use primitives::*;
|
2023-01-20 11:00:18 -05:00
|
|
|
|
|
|
|
|
#[derive(Clone, Copy, Encode, RuntimeDebug)]
|
2023-03-26 08:43:01 -04:00
|
|
|
#[cfg_attr(feature = "std", derive(scale::Decode, thiserror::Error))]
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
pub enum PalletError {
|
|
|
|
|
#[cfg_attr(feature = "std", error("batch for unrecognized network"))]
|
|
|
|
|
UnrecognizedNetwork,
|
|
|
|
|
#[cfg_attr(feature = "std", error("invalid signature for batch"))]
|
|
|
|
|
InvalidSignature,
|
2023-01-20 11:00:18 -05:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
#[frame_support::pallet]
|
|
|
|
|
pub mod pallet {
|
2023-11-05 20:02:34 +03:00
|
|
|
use sp_std::vec;
|
2023-03-31 06:34:09 -04:00
|
|
|
use sp_application_crypto::RuntimePublic;
|
2023-08-14 18:57:38 +03:00
|
|
|
use sp_runtime::traits::Zero;
|
|
|
|
|
use sp_core::sr25519::Public;
|
2023-03-31 06:34:09 -04:00
|
|
|
|
2023-11-05 20:02:34 +03:00
|
|
|
use serai_primitives::{Coin, SubstrateAmount, Amount, Balance};
|
|
|
|
|
|
2023-01-20 11:00:18 -05:00
|
|
|
use frame_support::pallet_prelude::*;
|
2023-11-05 20:02:34 +03:00
|
|
|
use frame_system::{pallet_prelude::*, RawOrigin};
|
2023-01-20 11:00:18 -05:00
|
|
|
|
2023-11-05 20:02:34 +03:00
|
|
|
use coins_pallet::{
|
|
|
|
|
Config as CoinsConfig, Pallet as Coins,
|
|
|
|
|
primitives::{OutInstruction, OutInstructionWithBalance},
|
|
|
|
|
};
|
|
|
|
|
use dex_pallet::{Config as DexConfig, Pallet as Dex};
|
2023-03-31 06:34:09 -04:00
|
|
|
use validator_sets_pallet::{
|
|
|
|
|
primitives::{Session, ValidatorSet},
|
|
|
|
|
Config as ValidatorSetsConfig, Pallet as ValidatorSets,
|
|
|
|
|
};
|
2023-01-28 01:47:13 -05:00
|
|
|
|
2023-01-20 11:00:18 -05:00
|
|
|
use super::*;
|
|
|
|
|
|
|
|
|
|
#[pallet::config]
|
2023-11-05 20:02:34 +03:00
|
|
|
pub trait Config:
|
|
|
|
|
frame_system::Config
|
|
|
|
|
+ CoinsConfig
|
|
|
|
|
+ DexConfig<MultiCoinId = Coin, CoinBalance = SubstrateAmount>
|
|
|
|
|
+ ValidatorSetsConfig
|
|
|
|
|
{
|
2023-01-20 11:00:18 -05:00
|
|
|
type RuntimeEvent: From<Event<Self>> + IsType<<Self as frame_system::Config>::RuntimeEvent>;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
#[pallet::event]
|
|
|
|
|
#[pallet::generate_deposit(fn deposit_event)]
|
|
|
|
|
pub enum Event<T: Config> {
|
2023-09-29 03:51:01 -04:00
|
|
|
Batch { network: NetworkId, id: u32, block: BlockHash, instructions_hash: [u8; 32] },
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
InstructionFailure { network: NetworkId, id: u32, index: u32 },
|
2023-01-20 11:00:18 -05:00
|
|
|
}
|
|
|
|
|
|
2023-11-05 20:02:34 +03:00
|
|
|
#[pallet::error]
|
|
|
|
|
pub enum Error<T> {
|
|
|
|
|
/// Coin and OutAddress types don't match.
|
|
|
|
|
InvalidAddressForCoin,
|
|
|
|
|
}
|
|
|
|
|
|
2023-01-20 11:00:18 -05:00
|
|
|
#[pallet::pallet]
|
|
|
|
|
pub struct Pallet<T>(PhantomData<T>);
|
|
|
|
|
|
2023-08-26 21:36:13 -04:00
|
|
|
// The ID of the last executed Batch for a network.
|
2023-01-20 11:00:18 -05:00
|
|
|
#[pallet::storage]
|
2023-04-16 02:57:19 -04:00
|
|
|
#[pallet::getter(fn batches)]
|
2023-08-26 21:36:13 -04:00
|
|
|
pub(crate) type LastBatch<T: Config> = StorageMap<_, Blake2_256, NetworkId, u32, OptionQuery>;
|
2023-01-20 11:00:18 -05:00
|
|
|
|
2023-08-14 18:57:38 +03:00
|
|
|
// The last Serai block in which this validator set included a batch
|
|
|
|
|
#[pallet::storage]
|
|
|
|
|
#[pallet::getter(fn last_batch_block)]
|
|
|
|
|
pub(crate) type LastBatchBlock<T: Config> =
|
|
|
|
|
StorageMap<_, Blake2_256, NetworkId, BlockNumberFor<T>, OptionQuery>;
|
|
|
|
|
|
2023-04-16 02:57:19 -04:00
|
|
|
// The latest block a network has acknowledged as finalized
|
|
|
|
|
#[pallet::storage]
|
2023-08-14 18:57:38 +03:00
|
|
|
#[pallet::getter(fn latest_network_block)]
|
|
|
|
|
pub(crate) type LatestNetworkBlock<T: Config> =
|
2023-04-16 02:57:19 -04:00
|
|
|
StorageMap<_, Blake2_256, NetworkId, BlockHash, OptionQuery>;
|
|
|
|
|
|
2023-01-28 01:47:13 -05:00
|
|
|
impl<T: Config> Pallet<T> {
|
2023-10-22 05:46:03 -04:00
|
|
|
// Use a dedicated transaction layer when executing this InInstruction
|
|
|
|
|
// This lets it individually error without causing any storage modifications
|
|
|
|
|
#[frame_support::transactional]
|
|
|
|
|
fn execute(instruction: InInstructionWithBalance) -> Result<(), DispatchError> {
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
match instruction.instruction {
|
2023-10-19 13:22:21 +03:00
|
|
|
InInstruction::Transfer(address) => {
|
2023-10-22 05:46:03 -04:00
|
|
|
Coins::<T>::mint(address.into(), instruction.balance)?;
|
2023-10-19 13:22:21 +03:00
|
|
|
}
|
2023-11-05 20:02:34 +03:00
|
|
|
InInstruction::Dex(call) => {
|
|
|
|
|
// This will only be initiated by external chain transactions. That is why we only need
|
|
|
|
|
// add liquidity and swaps. Other functionalities (such as remove_liq, etc) will be
|
|
|
|
|
// called directly from Serai with a native transaction.
|
|
|
|
|
match call {
|
|
|
|
|
DexCall::SwapAndAddLiquidity(address) => {
|
|
|
|
|
let origin = RawOrigin::Signed(IN_INSTRUCTION_EXECUTOR.into());
|
|
|
|
|
let coin = instruction.balance.coin;
|
|
|
|
|
|
|
|
|
|
// mint the given coin on the account
|
|
|
|
|
Coins::<T>::mint(IN_INSTRUCTION_EXECUTOR.into(), instruction.balance)?;
|
|
|
|
|
|
|
|
|
|
// swap half of it for SRI
|
|
|
|
|
let half = instruction.balance.amount.0 / 2;
|
|
|
|
|
let path = BoundedVec::try_from(vec![coin, Coin::Serai]).unwrap();
|
|
|
|
|
Dex::<T>::swap_exact_tokens_for_tokens(
|
|
|
|
|
origin.clone().into(),
|
|
|
|
|
path,
|
|
|
|
|
half,
|
|
|
|
|
1, // minimum out, so we accept whatever we get.
|
|
|
|
|
IN_INSTRUCTION_EXECUTOR.into(),
|
|
|
|
|
)?;
|
|
|
|
|
|
|
|
|
|
// get how much we got for our swap
|
|
|
|
|
let sri_amount = Coins::<T>::balance(IN_INSTRUCTION_EXECUTOR.into(), Coin::Serai).0;
|
|
|
|
|
|
|
|
|
|
// add liquidity
|
|
|
|
|
Dex::<T>::add_liquidity(
|
|
|
|
|
origin.clone().into(),
|
|
|
|
|
coin,
|
|
|
|
|
Coin::Serai,
|
|
|
|
|
half,
|
|
|
|
|
sri_amount,
|
|
|
|
|
1,
|
|
|
|
|
1,
|
|
|
|
|
address.into(),
|
|
|
|
|
)?;
|
|
|
|
|
|
|
|
|
|
// TODO: minimums are set to 1 above to guarantee successful adding liq call.
|
|
|
|
|
// Ideally we either get this info from user or send the leftovers back to user.
|
|
|
|
|
// Let's send the leftovers back to user for now.
|
|
|
|
|
let coin_balance = Coins::<T>::balance(IN_INSTRUCTION_EXECUTOR.into(), coin);
|
|
|
|
|
let sri_balance = Coins::<T>::balance(IN_INSTRUCTION_EXECUTOR.into(), Coin::Serai);
|
|
|
|
|
if coin_balance != Amount(0) {
|
|
|
|
|
Coins::<T>::transfer_internal(
|
|
|
|
|
IN_INSTRUCTION_EXECUTOR.into(),
|
|
|
|
|
address.into(),
|
|
|
|
|
Balance { coin, amount: coin_balance },
|
|
|
|
|
)?;
|
|
|
|
|
}
|
|
|
|
|
if sri_balance != Amount(0) {
|
|
|
|
|
Coins::<T>::transfer_internal(
|
|
|
|
|
IN_INSTRUCTION_EXECUTOR.into(),
|
|
|
|
|
address.into(),
|
|
|
|
|
Balance { coin: Coin::Serai, amount: sri_balance },
|
|
|
|
|
)?;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
DexCall::Swap(out_balance, out_address) => {
|
|
|
|
|
let send_to_external = !out_address.is_native();
|
|
|
|
|
let native_coin = out_balance.coin.is_native();
|
|
|
|
|
|
|
|
|
|
// we can't send native coin to external chain
|
|
|
|
|
if native_coin && send_to_external {
|
|
|
|
|
Err(Error::<T>::InvalidAddressForCoin)?;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
// mint the given coin on our account
|
|
|
|
|
Coins::<T>::mint(IN_INSTRUCTION_EXECUTOR.into(), instruction.balance)?;
|
|
|
|
|
|
|
|
|
|
// get the path
|
|
|
|
|
let mut path = vec![instruction.balance.coin, Coin::Serai];
|
|
|
|
|
if !native_coin {
|
|
|
|
|
path.push(out_balance.coin);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
// get the swap address
|
|
|
|
|
// if the address is internal, we can directly swap to it. if not, we swap to
|
|
|
|
|
// ourselves and burn the coins to send them back on the external chain.
|
|
|
|
|
let send_to = if send_to_external {
|
|
|
|
|
IN_INSTRUCTION_EXECUTOR
|
|
|
|
|
} else {
|
|
|
|
|
out_address.clone().as_native().unwrap()
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
// do the swap
|
|
|
|
|
let origin = RawOrigin::Signed(IN_INSTRUCTION_EXECUTOR.into());
|
|
|
|
|
Dex::<T>::swap_exact_tokens_for_tokens(
|
|
|
|
|
origin.into(),
|
|
|
|
|
BoundedVec::try_from(path).unwrap(),
|
|
|
|
|
instruction.balance.amount.0,
|
|
|
|
|
out_balance.amount.0,
|
|
|
|
|
send_to.into(),
|
|
|
|
|
)?;
|
|
|
|
|
|
|
|
|
|
// burn the received coins so that they sent back to the user
|
|
|
|
|
// if it is requested to an external address.
|
|
|
|
|
if send_to_external {
|
|
|
|
|
// see how much we got
|
|
|
|
|
let coin_balance =
|
|
|
|
|
Coins::<T>::balance(IN_INSTRUCTION_EXECUTOR.into(), out_balance.coin);
|
|
|
|
|
let instruction = OutInstructionWithBalance {
|
|
|
|
|
instruction: OutInstruction {
|
|
|
|
|
address: out_address.as_external().unwrap(),
|
|
|
|
|
// TODO: Properly pass data. Replace address with an OutInstruction entirely?
|
|
|
|
|
data: None,
|
|
|
|
|
},
|
|
|
|
|
balance: Balance { coin: out_balance.coin, amount: coin_balance },
|
|
|
|
|
};
|
|
|
|
|
Coins::<T>::burn_non_sri(IN_INSTRUCTION_EXECUTOR.into(), instruction)?;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
2023-01-28 01:47:13 -05:00
|
|
|
}
|
2023-10-22 05:46:03 -04:00
|
|
|
Ok(())
|
2023-01-28 01:47:13 -05:00
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2023-10-11 02:10:35 -04:00
|
|
|
fn keys_for_network<T: Config>(
|
|
|
|
|
network: NetworkId,
|
|
|
|
|
) -> Result<(Session, Option<Public>, Option<Public>), InvalidTransaction> {
|
2023-10-21 20:06:53 -04:00
|
|
|
// If there's no session set, and therefore no keys set, then this must be an invalid signature
|
|
|
|
|
let Some(session) = ValidatorSets::<T>::session(network) else {
|
|
|
|
|
Err(InvalidTransaction::BadProof)?
|
|
|
|
|
};
|
2023-08-14 18:57:38 +03:00
|
|
|
let mut set = ValidatorSet { session, network };
|
2023-10-10 23:55:59 -04:00
|
|
|
let latest = ValidatorSets::<T>::keys(set).map(|keys| keys.0);
|
|
|
|
|
let prior = if set.session.0 != 0 {
|
2023-08-14 18:57:38 +03:00
|
|
|
set.session.0 -= 1;
|
2023-10-10 23:55:59 -04:00
|
|
|
ValidatorSets::<T>::keys(set).map(|keys| keys.0)
|
|
|
|
|
} else {
|
|
|
|
|
None
|
|
|
|
|
};
|
|
|
|
|
if prior.is_none() && latest.is_none() {
|
|
|
|
|
Err(InvalidTransaction::BadProof)?;
|
2023-08-14 18:57:38 +03:00
|
|
|
}
|
2023-10-10 23:55:59 -04:00
|
|
|
Ok((session, prior, latest))
|
2023-08-14 18:57:38 +03:00
|
|
|
}
|
|
|
|
|
|
2023-01-20 11:00:18 -05:00
|
|
|
#[pallet::call]
|
|
|
|
|
impl<T: Config> Pallet<T> {
|
|
|
|
|
#[pallet::call_index(0)]
|
2023-01-28 01:47:13 -05:00
|
|
|
#[pallet::weight((0, DispatchClass::Operational))] // TODO
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
pub fn execute_batch(origin: OriginFor<T>, batch: SignedBatch) -> DispatchResult {
|
2023-01-20 11:00:18 -05:00
|
|
|
ensure_none(origin)?;
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
|
2023-08-14 18:57:38 +03:00
|
|
|
let batch = batch.batch;
|
|
|
|
|
|
|
|
|
|
LatestNetworkBlock::<T>::insert(batch.network, batch.block);
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
Self::deposit_event(Event::Batch {
|
|
|
|
|
network: batch.network,
|
|
|
|
|
id: batch.id,
|
|
|
|
|
block: batch.block,
|
2023-09-29 03:51:01 -04:00
|
|
|
instructions_hash: blake2_256(&batch.instructions.encode()),
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
});
|
2023-08-14 18:57:38 +03:00
|
|
|
for (i, instruction) in batch.instructions.into_iter().enumerate() {
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
if Self::execute(instruction).is_err() {
|
|
|
|
|
Self::deposit_event(Event::InstructionFailure {
|
|
|
|
|
network: batch.network,
|
|
|
|
|
id: batch.id,
|
|
|
|
|
index: u32::try_from(i).unwrap(),
|
|
|
|
|
});
|
2023-01-20 11:00:18 -05:00
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
Ok(())
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
#[pallet::validate_unsigned]
|
|
|
|
|
impl<T: Config> ValidateUnsigned for Pallet<T> {
|
2023-01-20 11:00:18 -05:00
|
|
|
type Call = Call<T>;
|
|
|
|
|
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
fn validate_unsigned(_: TransactionSource, call: &Self::Call) -> TransactionValidity {
|
2023-01-20 11:00:18 -05:00
|
|
|
// Match to be exhaustive
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
let batch = match call {
|
|
|
|
|
Call::execute_batch { ref batch } => batch,
|
2023-05-13 02:02:47 -04:00
|
|
|
Call::__Ignore(_, _) => unreachable!(),
|
2023-01-20 11:00:18 -05:00
|
|
|
};
|
|
|
|
|
|
2023-08-14 18:57:38 +03:00
|
|
|
// verify the batch size
|
|
|
|
|
// TODO: Merge this encode with the one done by batch_message
|
|
|
|
|
if batch.batch.encode().len() > MAX_BATCH_SIZE {
|
|
|
|
|
Err(InvalidTransaction::ExhaustsResources)?;
|
|
|
|
|
}
|
2023-01-20 11:00:18 -05:00
|
|
|
|
2023-10-11 02:10:35 -04:00
|
|
|
let network = batch.batch.network;
|
2023-10-22 05:38:44 -04:00
|
|
|
// Don't allow the Serai set to publish `Batch`s as-if Serai itself was an external network
|
|
|
|
|
if network == NetworkId::Serai {
|
|
|
|
|
Err(InvalidTransaction::Custom(0))?;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
// verify the signature
|
2023-10-11 02:10:35 -04:00
|
|
|
let (current_session, prior, current) = keys_for_network::<T>(network)?;
|
2023-10-13 12:14:59 -04:00
|
|
|
let batch_message = batch_message(&batch.batch);
|
2023-10-10 23:55:59 -04:00
|
|
|
// Check the prior key first since only a single `Batch` (the last one) will be when prior is
|
|
|
|
|
// Some yet prior wasn't the signing key
|
2023-10-13 12:14:59 -04:00
|
|
|
let valid_by_prior =
|
|
|
|
|
if let Some(key) = prior { key.verify(&batch_message, &batch.signature) } else { false };
|
2023-10-11 02:10:35 -04:00
|
|
|
let valid = valid_by_prior ||
|
|
|
|
|
(if let Some(key) = current {
|
2023-10-13 12:14:59 -04:00
|
|
|
key.verify(&batch_message, &batch.signature)
|
2023-10-11 02:10:35 -04:00
|
|
|
} else {
|
|
|
|
|
false
|
|
|
|
|
});
|
2023-10-10 23:55:59 -04:00
|
|
|
if !valid {
|
2023-03-31 06:34:09 -04:00
|
|
|
Err(InvalidTransaction::BadProof)?;
|
|
|
|
|
}
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
|
2023-10-10 23:55:59 -04:00
|
|
|
// If it wasn't valid by the prior key, meaning it was valid by the current key, the current
|
|
|
|
|
// key is publishing `Batch`s. This should only happen once the current key has verified all
|
|
|
|
|
// `Batch`s published by the prior key, meaning they are accepting the hand-over.
|
|
|
|
|
if prior.is_some() && (!valid_by_prior) {
|
2023-10-14 16:47:25 -04:00
|
|
|
ValidatorSets::<T>::retire_set(ValidatorSet {
|
|
|
|
|
network,
|
|
|
|
|
session: Session(current_session.0 - 1),
|
|
|
|
|
});
|
2023-10-10 23:55:59 -04:00
|
|
|
}
|
|
|
|
|
|
2023-08-14 18:57:38 +03:00
|
|
|
// check that this validator set isn't publishing a batch more than once per block
|
|
|
|
|
let current_block = <frame_system::Pallet<T>>::block_number();
|
|
|
|
|
let last_block = LastBatchBlock::<T>::get(network).unwrap_or(Zero::zero());
|
|
|
|
|
if last_block >= current_block {
|
|
|
|
|
Err(InvalidTransaction::Future)?;
|
|
|
|
|
}
|
2023-10-22 05:48:23 -04:00
|
|
|
LastBatchBlock::<T>::insert(batch.batch.network, frame_system::Pallet::<T>::block_number());
|
2023-08-14 18:57:38 +03:00
|
|
|
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
// Verify the batch is sequential
|
2023-08-26 21:36:13 -04:00
|
|
|
// LastBatch has the last ID set. The next ID should be it + 1
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
// If there's no ID, the next ID should be 0
|
2023-08-26 21:36:13 -04:00
|
|
|
let expected = LastBatch::<T>::get(network).map_or(0, |prev| prev + 1);
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
if batch.batch.id < expected {
|
|
|
|
|
Err(InvalidTransaction::Stale)?;
|
|
|
|
|
}
|
|
|
|
|
if batch.batch.id > expected {
|
|
|
|
|
Err(InvalidTransaction::Future)?;
|
|
|
|
|
}
|
2023-10-22 05:48:23 -04:00
|
|
|
LastBatch::<T>::insert(batch.batch.network, batch.batch.id);
|
2023-01-20 11:00:18 -05:00
|
|
|
|
2023-10-22 05:37:23 -04:00
|
|
|
// Verify all Balances in this Batch are for this network
|
|
|
|
|
for instruction in &batch.batch.instructions {
|
|
|
|
|
// Verify this coin is for this network
|
|
|
|
|
// If this is ever hit, it means the validator set has turned malicious and should be fully
|
|
|
|
|
// slashed
|
|
|
|
|
// Because we have an error here, no validator set which turns malicious should execute
|
|
|
|
|
// this code path
|
|
|
|
|
// Accordingly, there's no value in writing code to fully slash the network, when such an
|
|
|
|
|
// even would require a runtime upgrade to fully resolve anyways
|
|
|
|
|
if instruction.balance.coin.network() != batch.batch.network {
|
2023-10-22 05:38:44 -04:00
|
|
|
Err(InvalidTransaction::Custom(1))?;
|
2023-10-22 05:37:23 -04:00
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
Move in instructions from inherent transactions to unsigned transactions
The original intent was to use inherent transactions to prevent needing to vote
on-chain, which would spam the chain with worthless votes. Inherent
transactions, and our Tendermint library, would use the BFT's processs voting
to also vote on all included transactions. This perfectly collapses integrity
voting creating *no additional on-chain costs*.
Unfortunately, this led to issues such as #6, along with questions of validator
scalability when all validators are expencted to participate in consensus (in
order to vote on if the included instructions are valid). This has been
summarized in #241.
With this change, we can remove Tendermint from Substrate. This greatly
decreases our complexity. While I'm unhappy with the amount of time spent on
it, just to reach this conclusion, thankfully tendermint-machine itself is
still usable for #163. This also has reached a tipping point recently as the
polkadot-v0.9.40 branch of substrate changed how syncing works, requiring
further changes to sc-tendermint. These have no value if we're just going to
get rid of it later, due to fundamental design issues, yet I would like to
keep Substrate updated.
This should be followed by moving back to GRANDPA, enabling closing most open
Tendermint issues.
Please note the current in-instructions-pallet does not actually verify the
included signature yet. It's marked TODO, despite this bing critical.
2023-03-26 02:58:04 -04:00
|
|
|
ValidTransaction::with_tag_prefix("in-instructions")
|
|
|
|
|
.and_provides((batch.batch.network, batch.batch.id))
|
|
|
|
|
// Set a 10 block longevity, though this should be included in the next block
|
|
|
|
|
.longevity(10)
|
|
|
|
|
.propagate(true)
|
|
|
|
|
.build()
|
2023-01-20 11:00:18 -05:00
|
|
|
}
|
2023-10-13 00:31:23 -04:00
|
|
|
|
|
|
|
|
// Explicitly provide a pre-dispatch which calls validate_unsigned
|
|
|
|
|
fn pre_dispatch(call: &Self::Call) -> Result<(), TransactionValidityError> {
|
|
|
|
|
Self::validate_unsigned(TransactionSource::InBlock, call).map(|_| ()).map_err(Into::into)
|
|
|
|
|
}
|
2023-01-20 11:00:18 -05:00
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
pub use pallet::*;
|