EDCD / EDMarketConnector

Downloads commodity market and other station data from the game Elite: Dangerous for use with all popular online and offline trading tools.
GNU General Public License v2.0
991 stars 155 forks source link

Minor ResourceWarning in Inara plugin, unclosed SSLSocket #1267

Closed chennin closed 9 months ago

chennin commented 3 years ago

Running on 32-bit Python 3.9.6 on Windows, from the zip download and via Python, not the exe. ie:

C:\Users\Chris\Downloads\EDMarketConnector-stable>py -V
Python 3.9.6
C:\Users\Chris\Downloads\EDMarketConnector-stable>py -W all EDMarketConnector.py 1>EDMC.log 2>&1

EDMarketConnector-debug.log EDMC.log

Describe the bug

ResourceWarning on shutdown from the Inara plugin (I assume). Minor, but was asked to open the issue.

sys:1: ResourceWarning: unclosed <ssl.SSLSocket fd=2144, family=AddressFamily.AF_INET6, type=SocketKind.SOCK_STREAM, proto=0, laddr=('2601:18f:900:653:a88d:788:f25a:fd2', 54910, 0, 0), raddr=('2a01:28:ca:112::1:832', 443, 0, 0)>
$ dig +short aaaa inara.cz
2a01:28:ca:112::1:832

To Reproduce

  1. Start EDMC
  2. (I think this is needed) Start the game and log in
  3. Close EDMC
A-UNDERSCORE-D commented 3 years ago

Yeah its on the list, plugins dont clean up their messes super well

Rixxan commented 10 months ago

Hello @chennin!

I know it's been quite a while since you opened this issue, but as one of the new maintainers I'm trying to take triage of various issues. I can't seem to recreate this error on 5.10.0, and I was wondering if it still appears to occur for you, or what steps or customizations may be needed to a default test setup to recreate this issue.

Thanks!

chennin commented 9 months ago

I tried to retest and I don't seem to have the problem anymore, on v5.10.1+UNKNOWN (downloaded Windows zip from github) Running on Python v3.10.4. But, these days I am both on Linux primarily and not playing much Elite.

Rixxan commented 9 months ago

Thanks for the update. Given that, I'm going to close this one out.

Of course, if someone does manage to recreate it, please reopen this report so we can take a look at it!