[tor-bugs] #21919 [Core Tor/Tor]: hs: Change trunnel prop224 cell's namespace
Tor Bug Tracker & Wiki
blackhole at torproject.org
Tue Apr 11 14:39:02 UTC 2017
#21919: hs: Change trunnel prop224 cell's namespace
-------------------------------+---------------------------------------
Reporter: dgoulet | Owner: dgoulet
Type: defect | Status: new
Priority: Medium | Milestone: Tor: 0.3.1.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Keywords: tor-hs, refactor, prop224
Actual Points: | Parent ID: #21888
Points: 0.2 | Reviewer:
Sponsor: SponsorR-must |
-------------------------------+---------------------------------------
Currently, the trunnel namespace for hidden service cells (in
`src/trunnel/hs/`) is prefixed with `hs_cell_*`. We want to change this
for two reasons.
First, if we could have something in the name indicating that it is
trunnel, it would make it better for code semantic and separation.
Second, we want to create `hs_cells.[ch]` so we can put in there the cell
creation/parsing/handling instead of growing `hs_circuit.c` to the "hydra
size".
So for the renaming, here are some suggestions:
1. `tr_cell_*`
2. `tr_hs_cell_*`
3. `trunnel_cell_*`
Considering that an `ESTABLISH_INTRO` or `INTRODUCE1` cell is only for
hidden service, probably the `hs` in there is superfluous?
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21919>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list