From 7714902133496df29a46dc2001d81a8e069031a8 Mon Sep 17 00:00:00 2001 From: Max Inden Date: Sat, 20 Nov 2021 13:04:26 +0100 Subject: [PATCH] relay/DCUtR: Make message type field `required` The `type` field of the DCUtR message definition should not be `optional`, as a message without a `Type` is useless. Instead it should be `required`. See also circuit relay v2 specification using `required` for `type`. ```protobuf message HopMessage { enum Type { RESERVE = 0; CONNECT = 1; STATUS = 2; } required Type type = 1; ``` ```protobuf message StopMessage { enum Type { CONNECT = 0; STATUS = 1; } required Type type = 1; ``` --- relay/DCUtR.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/relay/DCUtR.md b/relay/DCUtR.md index dfa3ffc1b..81836c718 100644 --- a/relay/DCUtR.md +++ b/relay/DCUtR.md @@ -2,7 +2,7 @@ | Lifecycle Stage | Maturity | Status | Latest Revision | |-----------------|---------------|--------|--------------------| -| 1A | Working Draft | Active | r0, 2021-08-17 | +| 1A | Working Draft | Active | r1, 2021-11-20 | Authors: [@vyzo] @@ -135,7 +135,7 @@ message HolePunch { SYNC = 300; } - optional Type type=1; + required Type type=1; repeated bytes ObsAddrs = 2; }