pezkuwichain/pezkuwi-subxt
1
0
Fork 0
mirror of https://github.com/pezkuwichain/pezkuwi-subxt.git synced 2026-05-31 19:11:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Limit the maximum size of a downward message (#1690)

Browse Source
This commit is contained in:
Sergei Shulepov
2020-09-10 17:39:55 +02:00
committed by GitHub
parent f3abba15c6
commit e40ec81685
3 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
polkadot/roadmap/implementers-guide/src/runtime/router.md
+1
View File
@@ -260,6 +260,7 @@ any of dispatchables return an error.
Utility routines. Utility routines.
`queue_downward_message(P: ParaId, M: DownwardMessage)`: `queue_downward_message(P: ParaId, M: DownwardMessage)`:
1. Check if the serialized size of `M` exceeds the `config.critical_downward_message_size`. If so, return an error.
1. Wrap `M` into `InboundDownwardMessage` using the current block number for `sent_at`. 1. Wrap `M` into `InboundDownwardMessage` using the current block number for `sent_at`.
1. Obtain a new MQC link for the resulting `InboundDownwardMessage` and replace `DownwardMessageQueueHeads` for `P` with the resulting hash. 1. Obtain a new MQC link for the resulting `InboundDownwardMessage` and replace `DownwardMessageQueueHeads` for `P` with the resulting hash.
1. Add the resulting `InboundDownwardMessage` into `DownwardMessageQueues` for `P`. 1. Add the resulting `InboundDownwardMessage` into `DownwardMessageQueues` for `P`.
polkadot/roadmap/implementers-guide/src/types/messages.md
+2
View File
@@ -112,6 +112,8 @@ struct InboundHrmpMessage {
could be seen as a notification, however, it is conceivable that they might be used by the relay could be seen as a notification, however, it is conceivable that they might be used by the relay
chain to send a request to the parachain (likely, through the `ParachainSpecific` variant). chain to send a request to the parachain (likely, through the `ParachainSpecific` variant).
The serialized size of the message is limited by the `config.critical_downward_message_size` parameter.
```rust,ignore ```rust,ignore
enum DownwardMessage { enum DownwardMessage {
/// Some funds were transferred into the parachain's account. The hash is the identifier that /// Some funds were transferred into the parachain's account. The hash is the identifier that
polkadot/roadmap/implementers-guide/src/types/runtime.md
+7
View File
@@ -56,6 +56,13 @@ struct HostConfiguration {
pub dispatchable_upward_message_critical_weight: u32, pub dispatchable_upward_message_critical_weight: u32,
/// The maximum number of messages that a candidate can contain. /// The maximum number of messages that a candidate can contain.
pub max_upward_message_num_per_candidate: u32, pub max_upward_message_num_per_candidate: u32,
/// The maximum size of a message that can be put in a downward message queue.
///
/// Since we require receiving at least one DMP message the obvious upper bound of the size is
/// the PoV size. Of course, there is a lot of other different things that a parachain may
/// decide to do with its PoV so this value in practice will be picked as a fraction of the PoV
/// size.
pub critical_downward_message_size: u32,
/// Number of sessions after which an HRMP open channel request expires. /// Number of sessions after which an HRMP open channel request expires.
pub hrmp_open_request_ttl: u32, pub hrmp_open_request_ttl: u32,
/// The deposit that the sender should provide for opening an HRMP channel. /// The deposit that the sender should provide for opening an HRMP channel.