Skip to content

setConnection in example 5.5 may put UI in an inconsistent state #339

Closed

Description

@markafoltz

setConnection() calls close() on an existing connection before adding an onconnect handler to the new connection. If the close event on the old connection fires after the connect event on the new connection, reset() will null out the new connection and disable the stop and disconnect buttons.

Activity

  1. tidoust commented on Sep 26, 2016

    @tidoust
    Member

    See discussion at TPAC

    ACTION: @kenchris to review and fix the example in 5.5

  2. kenchris commented on Sep 30, 2016

    @kenchris
    Contributor
  3. markafoltz commented on Nov 16, 2016

    @markafoltz
    ContributorAuthor

    Closing with the fixes in the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions