search

koding / tunnel

Tunnel proxy package in Go
BSD 3-Clause "New" or "Revised" License
314 stars 70 forks source link

report fixes - continued #32

Closed mmatczuk closed 7 years ago

mmatczuk commented 7 years ago

This PR contains two left overs from https://github.com/koding/tunnel/pull/31. It leaves code with errchecks complaining only about Close and GoAway, and some minor issues in test.

@rjeczalik after updating the repo we need to press "Refresh" on https://goreportcard.com/report/github.com/koding/tunnel, I hope it can be done with web hook.

BTW the code just went from B to A class ;)

mmatczuk-stratoscale commented 7 years ago

@rjeczalik buffer write check was deleted thanks

rjeczalik commented 7 years ago

@mmatczuk-stratoscale: Thanks, looks good! :-)

mmatczuk commented 7 years ago

@rjeczalik I forgot to switch accounts....

rjeczalik commented 7 years ago

@mmatczuk Yup, I figured it was you :o)