[resteasy-commits] [resteasy/Resteasy] 4a2980: [RESTEASY-1838]: Remove the SseEventSource.open() ...

[GitHub]

  Branch: refs/heads/3.6
  Home:   https://github.com/resteasy/Resteasy
  Commit: 4a29803353d830a94e8054e0f5acb3e9a0213f29
      https://github.com/resteasy/Resteasy/commit/4a29803353d830a94e8054e0f5acb3e9a0213f29
  Author: NicoNes <nicolas.nesmon at gmail.com>
  Date:   2018-06-28 (Thu, 28 Jun 2018)

  Changed paths:
    M resteasy-client/src/main/java/org/jboss/resteasy/plugins/providers/sse/client/SseEventSourceImpl.java

  Log Message:
  -----------
  [RESTEASY-1838]: Remove the SseEventSource.open() 30 seconds timeout to be fully synch (#1535)

* Remove the 30 seconds timeout to be fully synch

Signed-off-by: NicoNes <nicolas.nesmon at gmail.com>

* Code cleaning:

- No need "disableKeepAlive" variable that is never set.
- Default close() method must wait up to 5 seconds instead of 30 seconds.

Signed-off-by: NicoNes <nicolas.nesmon at gmail.com>

* invoke `connectedLatch.countDown()` when the connection is established
instead of when the first SseEvent is received

Signed-off-by: NicoNes <nicolas.nesmon at gmail.com>



      **NOTE:** This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.