Merge pull request #124 from PepperDash/hotfix/ssh-autoreconnect-fix

Removes ReconnectTimer = null action from HandleConnectionFailure tim…
This commit is contained in:
Andrew Welker
2022-02-09 09:22:22 -07:00
committed by GitHub
3 changed files with 23 additions and 28 deletions

View File

@@ -21,7 +21,7 @@ env:
RELEASE_BRANCH: main RELEASE_BRANCH: main
jobs: jobs:
Build_Project: Build_Project:
runs-on: windows-latest runs-on: windows-2019
steps: steps:
# First we checkout the source repo # First we checkout the source repo
- name: Checkout repo - name: Checkout repo

View File

@@ -19,7 +19,7 @@ env:
RELEASE_BRANCH: main RELEASE_BRANCH: main
jobs: jobs:
Build_Project: Build_Project:
runs-on: windows-latest runs-on: windows-2019
steps: steps:
# First we checkout the source repo # First we checkout the source repo
- name: Checkout repo - name: Checkout repo

View File

@@ -319,28 +319,23 @@ namespace PepperDash.Core
/// </summary> /// </summary>
void HandleConnectionFailure() void HandleConnectionFailure()
{ {
KillClient(SocketStatus.SOCKET_STATUS_CONNECT_FAILED); KillClient(SocketStatus.SOCKET_STATUS_CONNECT_FAILED);
Debug.Console(1, this, "Client nulled due to connection failure. AutoReconnect: {0}, ConnectEnabled: {1}", AutoReconnect, ConnectEnabled);
Debug.Console(2, this, "Client nulled due to connection failure. AutoReconnect: {0}, ConnectEnabled: {1}", AutoReconnect, ConnectEnabled);
if (AutoReconnect && ConnectEnabled) if (AutoReconnect && ConnectEnabled)
{ {
Debug.Console(2, this, "Checking autoreconnect: {0}, {1}ms", Debug.Console(1, this, "Checking autoreconnect: {0}, {1}ms", AutoReconnect, AutoReconnectIntervalMs);
AutoReconnect, AutoReconnectIntervalMs); if (ReconnectTimer == null)
if (ReconnectTimer == null)// || !ReconnectTimerRunning)
{ {
ReconnectTimer = new CTimer(o => ReconnectTimer = new CTimer(o =>
{ {
Connect(); Connect();
ReconnectTimer = null;
}, AutoReconnectIntervalMs); }, AutoReconnectIntervalMs);
Debug.Console(1, this, "Attempting connection in {0} seconds", Debug.Console(1, this, "Attempting connection in {0} seconds",
(float)(AutoReconnectIntervalMs / 1000)); (float)(AutoReconnectIntervalMs / 1000));
} }
else else
{ {
Debug.Console(2, this, "{0} second reconnect cycle running", Debug.Console(1, this, "{0} second reconnect cycle running",
(float)(AutoReconnectIntervalMs / 1000)); (float)(AutoReconnectIntervalMs / 1000));
} }
} }
@@ -446,7 +441,7 @@ namespace PepperDash.Core
} }
else else
{ {
Debug.Console(2, this, "Client is null or disconnected. Cannot Send Text"); Debug.Console(1, this, "Client is null or disconnected. Cannot Send Text");
} }
} }
catch (Exception ex) catch (Exception ex)
@@ -478,7 +473,7 @@ namespace PepperDash.Core
} }
else else
{ {
Debug.Console(2, this, "Client is null or disconnected. Cannot Send Bytes"); Debug.Console(1, this, "Client is null or disconnected. Cannot Send Bytes");
} }
} }
catch catch