[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-bugs] #23360 [Core Tor/Tor]: prop224: Service has dead code which removed a feature
#23360: prop224: Service has dead code which removed a feature
------------------------------+--------------------------------
Reporter: dgoulet | Owner: dgoulet
Type: defect | Status: assigned
Priority: High | Milestone: Tor: 0.3.2.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Keywords: tor-hs, prop224
Actual Points: | Parent ID:
Points: | Reviewer:
Sponsor: |
------------------------------+--------------------------------
It appears that `hs_service_dir_info_changed()` has become unused after
the massive merge of #20657 which was replaced by
`hs_hsdir_set_changed_consider_reupload()` but then the
`consider_hsdir_upload_retry()` is not used anymore.
That feature was about retrying to HSDir for which when we want to upload
to, we are missing a descriptor. That is not possible to happen anymore
because in order to compute the HSDir index, the descriptor is needed thus
mandatory to have in order to learn if the HSDir is the correct one we
need on the hashring.
The fix here is to use `hs_service_dir_info_changed()` in `nodelist.c` and
make it do what `hs_hsdir_set_changed_consider_reupload()` does. And
remove the `hsdir_missing_info` feature.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/23360>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs