[tor-bugs] #1692 [Tor Relay]: No Events for SETCONF

Tor Bug Tracker & Wiki torproject-admin at torproject.org
Tue Aug 9 11:48:41 UTC 2011

#1692: No Events for SETCONF
 Reporter:  atagar       |          Owner:  krkhan            
     Type:  enhancement  |         Status:  assigned          
 Priority:  normal       |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor Relay    |        Version:                    
 Keywords:               |         Parent:                    
   Points:               |   Actualpoints:                    
Changes (by krkhan):

  * status:  needs_review => assigned


 Commit]: (Cherrypicked) Add smartlist_[v]asprintf_add
 Commit]: fixup! Add smartlist_[v]asprintf_add
 Commit]: Use smartlist_asprintf_add() to improve readability.
 Commit]: Escape configuration values before sending them via CONF_CHANGED.
 Commit]: Return if CONF_CHANGED isn't interesting.
  * '''[https://github.com/krkhan/tor/compare/bug1692 Tor patch]''':
 Complete patch

 > Also, please move the code that constructs a key-value list into
 > control_event_conf_changed -- that function should take as its
 > arguments the old and new or_options_t structures, and only make
 > a list of the options that changed if some control-port client
 > actually wants a CONF_CHANGED event.

  * Comparison of options is dependent on {{{option_is_same()}}} and
  * {{{option_is_same()}}} and {{{get_assigned_option()}}} require an
 argument of type {{{const config_format_t *}}}.
  * {{{config_format_t}}} is defined inside config.c, it needs to be
 defined in config.h along with other structs it's dependent on in order to
 be accessible from control.c

 I have refactored the code to make this work but since this generated a
 mammoth changeset, I have created a separate branch for this:

  * [https://github.com/krkhan/tor/compare/bug1692-refactored Alternate Tor

 Please let me know if I should continue with the refactored branch
 bug1692-refactored]) or the original branch
 ([https://github.com/krkhan/tor/tree/bug1692 bug1692]).

Ticket URL: <https://trac.torproject.org/projects/tor/ticket/1692#comment:20>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online

More information about the tor-bugs mailing list