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

Re: [tor-bugs] #9957 [Tor]: Tor should consider stderr output of transport proxies



#9957: Tor should consider stderr output of transport proxies
------------------------+--------------------------------
     Reporter:  wfn     |      Owner:
         Type:  defect  |     Status:  needs_revision
     Priority:  minor   |  Milestone:  Tor: 0.2.6.x-final
    Component:  Tor     |    Version:
   Resolution:          |   Keywords:  tor-pt
Actual Points:          |  Parent ID:
       Points:          |
------------------------+--------------------------------
Changes (by asn):

 * status:  needs_review => needs_revision


Comment:

 Replying to [comment:15 wfn]:
 > Rebased once again (same branch) - to include an additional change which
 includes the `report_proxy_stderr` keyword that a user can look for when
 inspecting INFO-level tor log:
 >
 > https://github.com/wfn/tor/compare/bug_9957_2 ->
 >
 >   *
 https://github.com/wfn/tor/commit/9b3ef629225fa3a0fea0a7090a3ee8a4b14ffab1
 >   *
 https://github.com/wfn/tor/commit/f8152a1fd36e5944ccf604a74bfbbb776e791786

 Nice. This looks cleaner!

 As a suggestion, to avoid needless nesting, maybe you would enjoy doing:
 {{{
 if (!proxy_err_output) {
     return
 }
 ... do everything ...
 }}}
 instead of:
 {{{
 if (proxy_err_output) {
       ... do everything ...
 }
 }}}
 Up to your preference :)

 Also, AFAIK, we are using `STATIC` to mean "We should use static here, but
 we also need to unit test that function". Since you are not unittesting
 `report_proxy_stderr()` the function can be truly `static` like other
 functions in `src/or/transports.c` (for example
 `handle_finished_proxy()`).

 Finally, we will need a changes file if we want this merged. I can write
 this for you, but you might want to write it on your own to get used to
 changes files for further Tor development :) As an example, see commit
 `71e0ca02b57f7945d922a8708a2c97815a9350ad`.

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