Opened 15 years ago

Last modified 7 years ago

#120 closed defect (Implemented)

servers don't transform destroys to truncateds

Reported by: arma Owned by:
Priority: Low Milestone:
Component: Core Tor/Tor Version: 0.1.0.1-rc
Severity: Keywords:
Cc: arma Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

In command_process_destroy_cell(), we are supposed to convert client-ward destroy cells into
truncated cells, so the client can know at what hop in the path the circuit broke. But we
don't actually do this currently, because the client behave was always to close the circuit,
so we were wasting bandwidth with a two-round-trip circuit close.

If we're going to start trying to re-extend circuits, though, we need this functionality.
Should we re-enable the ifdef 0'ed stuff (and remove the circuit_mark_for_close())?

[Automatically added by flyspray2trac: Operating System: All]

Child Tickets

Change History (3)

comment:1 Changed 15 years ago by nickm

Sounds quite reasonable to me. I don't think it will hurt anything, and right now I don't think that we're
often blocking on tearing circuits down, so the latency required to do so shouldn't be too high.

comment:2 Changed 15 years ago by arma

flyspray2trac: bug closed.
Ok, I did this. We'll see if it works.

comment:3 Changed 7 years ago by nickm

Component: Tor RelayTor
Note: See TracTickets for help on using tickets.