Fix: ResourceWarning unclosed socket

[rmkraeuchi]

Close socket if error occurs in open_socket. This can happen if websocket is not available.

Current Code:

url = 'ws://10.1.1.11:8081/ws'   # any unreachable url is working
ws = websocket.WebSocket()
ws.connect(url=url, timeout=0.1)

Received warning in console:
ResourceWarning: unclosed <socket.socket fd=636, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=6, laddr=('0.0.0.0', 56100), raddr=('10.1.1.11', 8081)>

[codecov]

Codecov Report

Merging #690 (f844681) into master (afd8a77) will increase coverage by 0.11%.
The diff coverage is 50.00%.

❗ Current head f844681 differs from pull request most recent head 2179eea. Consider uploading reports for the commit 2179eea to get more accurate results

@@            Coverage Diff             @@
##           master     #690      +/-   ##
==========================================
+ Coverage   74.72%   74.84%   +0.11%     
==========================================
  Files          13       13              
  Lines        1286     1288       +2     
  Branches      250      251       +1     
==========================================
+ Hits          961      964       +3     
+ Misses        211      209       -2     
- Partials      114      115       +1     

Impacted Files	Coverage Δ
websocket/_http.py	70.53% <50.00%> (-0.21%)	⬇️
websocket/_handshake.py	73.91% <0.00%> (-1.74%)	⬇️
websocket/_core.py	78.00% <0.00%> (+1.00%)	⬆️
websocket/_utils.py	88.88% <0.00%> (+5.55%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update afd8a77...2179eea. Read the comment docs.

[engn33r]

Thanks for the PR. Good catch, although I was not able to recreate the error message you received. I haven't checked if there are other places in this project where resources are not cleanly handled in error or exception cases.