[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-bugs] #33665 [Circumvention/Snowflake]: Add command-line options for unsafe logging



#33665: Add command-line options for unsafe logging
-----------------------------------------+----------------------
     Reporter:  dcf                      |      Owner:  arlolra
         Type:  enhancement              |     Status:  assigned
     Priority:  Medium                   |  Milestone:
    Component:  Circumvention/Snowflake  |    Version:
     Severity:  Normal                   |   Keywords:
Actual Points:                           |  Parent ID:
       Points:                           |   Reviewer:
      Sponsor:                           |
-----------------------------------------+----------------------
 Moved from comment:4:ticket:33157

 >>> Something like that patch was useful when working on #19026 so would
 you consider merging,
 https://github.com/keroserene/snowflake/commit/dbd733e4b1430c046ec11e8052efdbac6010e58a
 >>>
 >> It's okay with me but I would call the option --unsafe-logging instead
 of --unsafeLogging to match the style of the other options.
 >>
 > In `broker/broker.go`, yes, dashes are used (ex. `disable-tls`, `acme-
 email`, etc), however, in `client/snowflake.go` we have camel casing (ex.
 `logToStateDir`, `keepLocalAddresses`, etc)
 >
 > We should probably consolidate this now.  I imagine your preference is
 for the former?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/33665>
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