Skip to content

Make it possible to explicitly close RPC connection#150

Merged
petertodd merged 2 commits into
petertodd:masterfrom
simonmulser:master
Nov 6, 2017
Merged

Make it possible to explicitly close RPC connection#150
petertodd merged 2 commits into
petertodd:masterfrom
simonmulser:master

Conversation

@simonmulser

@simonmulser simonmulser commented Oct 11, 2017

Copy link
Copy Markdown
Contributor

So far it is not possible to close a connection. you need to hope that GC does it or access __connection over the __dict__ variable.

This PR let the proxy expose a proper close() function.

@petertodd petertodd merged commit 1ea9661 into petertodd:master Nov 6, 2017
petertodd added a commit that referenced this pull request Nov 6, 2017
1ea9661 make it possible to explicitly close connection (Simon Mulser)
4df1bbb fix PEP8 too many blank lines (Simon Mulser)

Pull request description:

  So far it is not possible to close a connection. you need to hope that GC does it or access `__connection` over the `__dict__` variable.

  This PR let the proxy expose a proper `close()` function.

Tree-SHA512: 721c4bd250549264ec54b3a2f8c333dcd55ed90cfc9ff7e8720c2243ada3639785889e52adc6a2fdf4b5c670bc54e3c127b57b04215cc119d419e79794107598
@petertodd

Copy link
Copy Markdown
Owner

Thanks!

rawBit-io pushed a commit to rawBit-io/python-bitcointx that referenced this pull request Nov 6, 2025
b74e9d0 make it possible to explicitly close connection (Simon Mulser)
ff08f9a fix PEP8 too many blank lines (Simon Mulser)

Pull request description:

  So far it is not possible to close a connection. you need to hope that GC does it or access `__connection` over the `__dict__` variable.

  This PR let the proxy expose a proper `close()` function.

Tree-SHA512: 721c4bd250549264ec54b3a2f8c333dcd55ed90cfc9ff7e8720c2243ada3639785889e52adc6a2fdf4b5c670bc54e3c127b57b04215cc119d419e79794107598
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants