Kurento / bugtracker

[ARCHIVED] Contents migrated to monorepo: https://github.com/Kurento/kurento
46 stars 10 forks source link

java-client 6.6.2: If server connection goes away, the connection cannot be closed by user #238

Open brstgt opened 6 years ago

brstgt commented 6 years ago

KMS Version: 6.6.2 Client libraries

What steps will reproduce the problem?

  1. Connect to server
  2. Shutdown server
  3. Client starts trying to reconnect
  4. call .destroy() on connection
  5. Reconnections continue and fail with exception+
  6. No chance to close the connection

throw new JsonRpcClientClosedException("Trying to send a message in a client closed explicitly. When a client is closed, it can't be reused. It is necessary to create another one");

AbstractJsonRpcClientWebSocket.reconnect should check for this.isClosedByUser()

What is the expected result? Manually destroying a connection should stop the connection retries

What happens instead? Endless connection retries

ghost commented 6 years ago

Hi brstgt, I have the same exception: Trying to send a message in a client closed explicitly. When a client is closed, it can't be reused. It is necessary to create another one and I'm trying to find out why it happens. Could you describe your case more detailed:

  1. In steps 1-2(connect/shoutdown server) do you mean kurento media server?
  2. 'call .destory()' does it mean KurentoClient.destroy()? I will be appreciative for your help.
aashirwadgupta commented 5 years ago

Did anyone find anything on this?

micaelgallego commented 5 years ago

Have you tested with latest version?

On Wed, Oct 9, 2019 at 3:01 PM Aashirwad Gupta notifications@github.com<mailto:notifications@github.com> wrote:

Did anyone find anything on this?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/Kurento/bugtracker/issues/238?email_source=notifications&email_token=AAIF2KHRRU6ZTRNKA7WFNVLQNXI35A5CNFSM4EXSLTO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAXZTOQ#issuecomment-539990458, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAIF2KFSOPAK3JPEFDXCVOLQNXI35ANCNFSM4EXSLTOQ.

aashirwadgupta commented 5 years ago

I do not have OpenVidu license so I am trying with version 6.10.0

micaelgallego commented 5 years ago

OpenVidu is open source. You don't need any license to use it.

Latest Kurento version is 6.11.0. We are working right now in publishing 6.12.0 release.

Regards

Micael Gallego https://gestion2.urjc.es/pdi/ver/micael.gallego ElasTest Project Technical Coordinator - http://elastest.io Profesor en la Escuela Técnica Superior de Ingeniería Informática URJC

On Thu, Oct 10, 2019 at 8:37 AM Aashirwad Gupta notifications@github.com wrote:

I do not have OpenVidu license so I am trying with version 6.10.0

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Kurento/bugtracker/issues/238?email_source=notifications&email_token=AAIF2KCZ53GB2JRVZKELCLTQN3EQ7A5CNFSM4EXSLTO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA27LDA#issuecomment-540407180, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIF2KDABSOKRU5YV7B4XCLQN3EQ7ANCNFSM4EXSLTOQ .