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

Re: [tor-bugs] #11183 [meek]: Make an HTTP requestor Firefox extension for meek-client



#11183: Make an HTTP requestor Firefox extension for meek-client
-------------------------+--------------------
     Reporter:  dcf      |      Owner:  dcf
         Type:  project  |     Status:  closed
     Priority:  normal   |  Milestone:
    Component:  meek     |    Version:
   Resolution:  fixed    |   Keywords:  meek
Actual Points:           |  Parent ID:  #10935
       Points:           |
-------------------------+--------------------

Comment (by dcf):

 Replying to [comment:20 gk]:
 > Review of https://gitweb.torproject.org/pluggable-
 transports/meek.git/blob/HEAD:/firefox/component/main.js part 1
 >
 > {{{
 >  4 // The extension opens a TCP socket listening on localhost (port
 7000).
 > }}}
 > See previous comment: you seem to advocate hard-coding a listener port,
 no?
 > {{{
 >  50 Components.interfaces.nsIServerSocketListener,
 > }}}
 > Nit: there is no "," needed here as this is the last interfaces listed.
 > {{{
 >  56 return
 > }}}
 > Nit: a ";" is missing at the end.
 > {{{
 >  124 this.requestreader = null;
 > }}}
 > Nit: should be "this.requestReader = null;" see line 131 as well.
 > {{{
 >  130 readRequest: function(callback) {
 > }}}
 > Seems you can omit the |callback|, no? You are not using it in the
 readRequest method. Or did you plan to pass it to the RequestReader
 constructor?

 I think these are all answered as of https://gitweb.torproject.org
 /pluggable-
 transports/meek.git/shortlog/eab44b4fbaf3aabc7077a06d92ee02ce61b57932. All
 but the trailing comma, as that's deliberate.

 > I have to think a bit about your usage of nsISocketTransport and whether
 you are affected by the things we ran into in #9531. See the comments
 there (e.g. regarding OPEN_BLOCKING|OPEN_UNBUFFERED)

 Okay. I don't know what to make of that.

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