Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SDLIAPTransport Does Not Reconnect Successfully #816

Closed
brandon-salahat-tm opened this issue Dec 7, 2017 · 0 comments
Closed

SDLIAPTransport Does Not Reconnect Successfully #816

brandon-salahat-tm opened this issue Dec 7, 2017 · 0 comments
Labels
bug A defect in the library
Milestone

Comments

@brandon-salahat-tm
Copy link
Contributor

brandon-salahat-tm commented Dec 7, 2017

Bug Report

The current IAP transport implementation often fails during the reconnection logic. Rapid disconnects/reconnects amplify the issue.

Reproduction Steps
  1. Connect a SDL app.
  2. Disconnect a SDL app, or initiate a transport mode switch if supported on the head unit
  3. Observe if app successfully reconnects
  4. Repeat steps 1-3
Expected Behavior

SDL app successfully connects

Observed Behavior

SDL apps often do not connect

This most often is due to two small logical issues inside the transport adapter

  1. When the proxy reconnects after a delay, it holds a reference to the existing accessory. In many cases the accessory could have disconnected and reconnected inside of that delay, and the referenced accessory becomes invalid. Attempting to connect to the invalid accessory causes connection failure.

  2. There is a race condition that occurs with the protocolIndexTimer which in many cases can cause the start selector to fire after the cancel selector.

OS & Version Information
  • iOS Version: iOS 10 and 11, including subversions.
  • SDL iOS Version: Issue observed in 5.0 release, and 4.7.3 release.
  • Testing Against: Head unit
Test Case, Sample Code, and / or Example App

PR 817

@joeljfischer joeljfischer added the bug A defect in the library label Dec 8, 2017
@joeljfischer joeljfischer added this to the 5.1.0 milestone Jan 9, 2018
@joeljfischer joeljfischer mentioned this issue Jan 10, 2018
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A defect in the library
Projects
None yet
Development

No branches or pull requests

2 participants