[tor-commits] [torspec] branch main updated: Document missing NL in the middle layer of an HsDesc.

[gitolite role]

This is an automated email from the git hooks/post-receive script.

dgoulet pushed a commit to branch main
in repository torspec.

The following commit(s) were added to refs/heads/main by this push:
     new b842f50  Document missing NL in the middle layer of an HsDesc.
     new f88f643  Merge branch 'tor-gitlab/mr/109'
b842f50 is described below

commit b842f50c4f952d9baec83395dd782241c20de610
Author: Nick Mathewson <nickm at torproject.org>
AuthorDate: Mon Jan 30 16:46:58 2023 -0500

    Document missing NL in the middle layer of an HsDesc.
    
    It looks like C tor doesn't include a final newline in the middle
    layer of its onion service descriptors.  That made arti reject them
    the first time I tried to parse one!  Here I document this behavior,
    and tell other implementations what to do.
---
 rend-spec-v3.txt | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/rend-spec-v3.txt b/rend-spec-v3.txt
index 278c1fa..a4a827a 100644
--- a/rend-spec-v3.txt
+++ b/rend-spec-v3.txt
@@ -1270,6 +1270,13 @@ Table of contents:
       discussed in [HS-DESC-SECOND-LAYER] below. The blob is base64 encoded
       and enclosed in -----BEGIN MESSAGE---- and ----END MESSAGE---- wrappers.
 
+
+    Compatibility note: The C Tor implementation does not include a final
+    newline when generating this first-layer-plaintext section; other
+    implementations MUST accept this section even if it is missing its final
+    newline.  Other implementations MAY generate this section without a final
+    newline themselves, to avoid being distinguishable from C tor.
+
 2.5.1.3. Client behavior [FIRST-LAYER-CLIENT-BEHAVIOR]
 
     The goal of clients at this stage is to decrypt the "encrypted" field as

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.