Opened 3 years ago

Last modified 8 months ago

#25654 new enhancement

Create a testing-only handshake for shaking the bugs out of wide create cells (prop249)

Reported by: nickm Owned by:
Priority: Medium Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-cell, tor-circuit, trunnel, 034-roadmap-subtask, 034-triage-20180328, 034-included-20180328, 035-roadmap-subtask, 035-triaged-in-20180711
Cc: dgoulet, nickm Actual Points:
Parent ID: #24986 Points:
Reviewer: Sponsor:


We need a handshake that doesn't fit in a single CREATE cell so that we can make sure that prop249 is implemented properly.

My current recommendation is "nnttoorr", which is the same as ntor, but every byte is repeated twice. ;)

Child Tickets

Change History (11)

comment:1 Changed 3 years ago by isis

Owner: set to isis
Status: newaccepted

Assigning the crazypants handshake to myself, since I'll need to touch this part of the codebase (or at least become super familiar with it) for #24988.

Also, depending on timing of when things are finished, it might be possible to use the work from #24987 (since it composes two handshakes by taking function pointers to the implementations) to chain ntor+ntor to accomplish this task… that would involve no (or very little) new code.

comment:2 Changed 3 years ago by nickm

Keywords: 034-roadmap-subtask added

comment:3 Changed 3 years ago by nickm

Keywords: 034-triage-20180328 added

comment:4 Changed 3 years ago by nickm

Keywords: 034-included-20180328 added

comment:5 Changed 2 years ago by isis

Type: defectenhancement

comment:6 Changed 2 years ago by isis

Milestone: Tor: 0.3.4.x-finalTor: 0.3.5.x-final

comment:7 Changed 2 years ago by nickm

Keywords: 035-roadmap-subtask added

comment:8 Changed 2 years ago by nickm

Keywords: 035-triaged-in-20180711 added

comment:9 Changed 2 years ago by nickm

Milestone: Tor: 0.3.5.x-finalTor: unspecified

We won't get PQ or wide-create done in this release. :/

comment:10 Changed 2 years ago by gaba

Cc: isis removed
Owner: isis deleted
Status: acceptedassigned

comment:11 Changed 8 months ago by teor

Status: assignednew

Change tickets that are assigned to nobody to "new".

Note: See TracTickets for help on using tickets.