search

colyseus / proxy

🔀⚔ Proxy and Service Discovery for Colyseus 0.10 ~ 0.14 (Not recommended on 0.15+)
https://docs.colyseus.io/scalability/
MIT License
37 stars 25 forks source link

fix proxy error ignore rule #22

Closed jldinh closed 2 years ago

jldinh commented 2 years ago

Hi, I spotted an issue in the ignore rule supposed to fix #14. The unregistration is supposed to happen only when the error is neither socket hang up nor ECONNRESET, as these are both transient errors. However, as the code is currently written, unregistration will always happen due to a typo in the condition.

endel commented 2 years ago

Thanks a lot for this @jldinh! Sorry for the delay, I'm just back from vacation!

jldinh commented 2 years ago

No problem!

gweeish commented 2 years ago

Hi @jldinh,

I'm so sorry but I'm not sure where else to ask this question.

I'm trying to install multicell and in the process, I'm trying to install OpenAlea. However, I'm having the following issue with the 'python multisetup.py develop --user'. The issue is as follows:

'Error processing line 1 of /Users/gweeish/.local/lib/python3.7/site-packages/OpenAlea.SConsX-nspkg.pth:

Traceback (most recent call last): File "/Users/gweeish/.pyenv/versions/3.7.5/lib/python3.7/site.py", line 168, in addpackage exec(line) File "", line 1, in File "", line 580, in module_from_spec AttributeError: 'NoneType' object has no attribute 'loader'

Remainder of file ignored Error processing line 1 of /Users/gweeish/.local/lib/python3.7/site-packages/openalea.deploy-nspkg.pth:

Traceback (most recent call last): File "/Users/gweeish/.pyenv/versions/3.7.5/lib/python3.7/site.py", line 168, in addpackage exec(line) File "", line 1, in File "", line 580, in module_from_spec AttributeError: 'NoneType' object has no attribute 'loader'

Remainder of file ignored Install OpenAlea.Deploy first Install OpenAlea.Deploy first cannot import name 'Multisetup' from 'multisetup' (/Users/gweeish/openalea/multisetup.py) cannot import name 'Multisetup' from 'multisetup' (/Users/gweeish/openalea/multisetup.py) Traceback (most recent call last): File "multisetup.py", line 73, in main() File "multisetup.py", line 68, in main mysetup = Multisetup(curdir='.', commands=args, packages=dirs) NameError: name 'Multisetup' is not defined'

Any help would be greatly appreciated.

Thank you!

jldinh commented 2 years ago

Hi @gweeish , sure we can have a look. Please go to https://github.com/jldinh/multicell/issues/1, and we'll continue the discussion there to avoid polluting this issue with unrelated content.