EST_INTRO_DOS_EXT: parameter validation
In torspec#167 torspec!88 we propose to tighten up the parsing of this cell.
FTR this is the currently behaviour in Arti (since !648 (merged)):
- We do not check that
rate <= burst
; probably this ought to be checked inEstIntroExtDoS::new
and also on cell parsing (theReadable
impl) - We accept repeated parameters, taking the last.
- We accept invalid parameters, discarding them (and using consensus default).
- We do not check parameter ordering when deserialising.
The current state of torpsec!88 would imply changing 1-3 and leaving 4 as it is.