Changes between Initial Version and Version 1 of Ticket #29077, comment 13


Ignore:
Timestamp:
Jan 26, 2019, 3:19:48 PM (5 months ago)
Author:
yawning
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #29077, comment 13

    initial v1  
    22
    33Looks ok at a first glance.  I guess I should add the timeout values to meek_lite too.
     4
     5Edit: Done, though I just copy the values out of http.DefaulTransport by hand instead of doing anything with reflection.
     6
     7As far as interoperability/compatibility goes, I'm open to changing meek_lite as well.
     8
     9Note that I treat not specifying the argument as "use a sensible default", and have `none` special cased as "use the built in TLS stack", under the rationale that once the code is solid the only reason to not use utls is if something breaks, and "Add `utls=none`" is a easier interim workaround to convey to the unwashed masses than "Edit an existing argument".
     10
     11Let me know if you would rather I change this to match meek's behavior.
     12
     13There's a number of other things that I'm tempted to do with the code (assuming I have the spare time) that would increase compatibility and blocking resistance, but it would require forking utls, so no promises.