Opened 14 months ago

Closed 8 months ago

#26840 closed enhancement (implemented)

prop289: Modify SENDME cell to have a version and payload

Reported by: dgoulet Owned by: dgoulet
Priority: Medium Milestone: Tor: 0.4.1.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: prop289, tor-spec, 035-roadmap-subtask, prop289-assigned-sponsor-v, 041-proposed-on-roadmap
Cc: Actual Points:
Parent ID: #26288 Points:
Reviewer: Sponsor: SponsorV

Description

To implement prop289, we need the SENDME cell (empty payload right now) to have a version and payload for the bytes inserted into them.

1) We should have a trunnel definition with a proper specification of the cell.

2) Have the code interface to construct those new SENDME cells but don't put them on the wire just yet.

For (2), we could put them on the wire right now but we should make sure tor will accept them (validating empty payload vs not looking at the payload at all).

Child Tickets

Change History (9)

comment:1 Changed 14 months ago by dgoulet

Keywords: 035-roadmap-subtask added
Sponsor: SponsorV

comment:2 Changed 14 months ago by dgoulet

Type: defectenhancement

comment:3 Changed 14 months ago by teor

Keywords: prop289-assigned-sponsor-v added

I think all prop#289 tickets are Sponsor V.

comment:4 Changed 13 months ago by dgoulet

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

Deferring prop289 work to 036

comment:5 Changed 11 months ago by nickm

Milestone: Tor: 0.3.6.x-finalTor: 0.4.0.x-final

Tor 0.3.6.x has been renamed to 0.4.0.x.

comment:6 Changed 9 months ago by dgoulet

Spec branch that details the new structure of the SENDME cell: ticket26840_01.

comment:7 Changed 9 months ago by dgoulet

Milestone: Tor: 0.4.0.x-finalTor: 0.4.1.x-final

prop289 won't make it in 040. Feature freeze is in 7 days.

comment:8 Changed 9 months ago by teor

Keywords: 041-proposed-on-roadmap added

Let's review these tickets at the next meeting using our 041-proposed process.
They're on the roadmap, so the review should focus on ticket size and team capacity (and sponsor expectations).

comment:9 Changed 8 months ago by nickm

Resolution: implemented
Status: assignedclosed

Done as part of parent.

Note: See TracTickets for help on using tickets.