#31684: Add control port GETINFO support for dumping the local consensus --------------------------------------------+------------------------------ Reporter: asn | Owner: (none) Type: enhancement | Status: merge_ready Priority: Medium | Milestone: Tor: | 0.4.3.x-final Component: Core Tor/Tor | Version: Severity: Normal | Resolution: Keywords: control-port easy extra-review | Actual Points: 0.5 Parent ID: | Points: 1 Reviewer: asn | Sponsor: --------------------------------------------+------------------------------ Changes (by teor):
* keywords: control-port easy extra-review nickm-merge dgoulet-merge => control-port easy extra-review * type: task => enhancement * actualpoints: => 0.5 Comment: I did some fixes: * standard whitespace * typos * standard error handling * remove unnecessary test code Here is the branch with all the fixes: https://github.com/teor2345/tor/commits/ticket31684 There is a description of each fix in my review on PR 1328. Here is the squashed PR for merging: * master: https://github.com/torproject/tor/pull/1434 I squashed the whitespace fixes with the rest of the code, to minimise the diff. Since these are obvious fixes on already reviewed code, any maintainer can merge after CI passes. -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/31684#comment:27> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs