search

fabric8io / mockwebserver

An extension of okhttp's mockwebserver, that provides a DSL and is easier to use
Apache License 2.0
113 stars 38 forks source link

Update okhttp to 3.9.0 #16

Closed luciofm closed 7 years ago

luciofm commented 7 years ago

Also fix a websocket close with an invalid reason code that was leading to a crash.

jimmidyson commented 7 years ago

@iocanel Are you ok with this upgrade? I vaguely remember we discussed this before and you wanted to stick to 3.6.0 to be consistent with other fabric8/syndesis libraries? I'm in favour of this upgrade personally and then upgrading other libraries that are dependent on this as needed.

luciofm commented 7 years ago

@jimmidyson didn't saw you comment earlier... Sent another commit updating to the latest OkHttp (3.9.0) and adding a test case for the websocket fix

iocanel commented 7 years ago

Ok, let's upgrade here and then also upgrade the kuberntes-client too.