Improve publish BadSessionIdInvalid and BadSecureChannelIdInvalid (#378)

* Handle more errors during publish * Update version
Azure-Samples · Jul 25, 2024 · 3328ca9 · 3328ca9
1 parent a4dfb11
commit 3328ca9
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 9 deletions.
diff --git a/src/PlcServer.cs b/src/PlcServer.cs
@@ -220,20 +220,40 @@ public override ResponseHeader Publish(
         {
             MetricsHelper.RecordTotalErrors(nameof(Publish));
 
+            subscriptionId = default;
+            availableSequenceNumbers = default;
+            moreNotifications = default;
+            notificationMessage = default;
+            results = default;
+            diagnosticInfos = default;
+
             if (ex.Message == StatusCodes.BadNoSubscription.ToString())
             {
-                _logger.LogDebug("Failed to publish: BadNoSubscription");
-
-                subscriptionId = default;
-                availableSequenceNumbers = default;
-                moreNotifications = default;
-                notificationMessage = default;
-                results = default;
-                diagnosticInfos = default;
+                _logger.LogDebug(
+                    "Failed to publish: {StatusCode}",
+                    StatusCodes.BadNoSubscription.ToString());
 
                 return new ResponseHeader { ServiceResult = StatusCodes.BadNoSubscription };
             }
 
+            if (ex.Message == StatusCodes.BadSessionIdInvalid.ToString())
+            {
+                _logger.LogDebug(
+                    "Failed to publish: {StatusCode}",
+                    StatusCodes.BadSessionIdInvalid.ToString());
+
+                return new ResponseHeader { ServiceResult = StatusCodes.BadSessionIdInvalid };
+            }
+
+            if (ex.Message == StatusCodes.BadSecureChannelIdInvalid.ToString())
+            {
+                _logger.LogDebug(
+                    "Failed to publish: {StatusCode}",
+                    StatusCodes.BadSecureChannelIdInvalid.ToString());
+
+                return new ResponseHeader { ServiceResult = StatusCodes.BadSecureChannelIdInvalid };
+            }
+
             _logger.LogError(ex, "Error publishing");
             throw;
         }

diff --git a/version.json b/version.json
@@ -1,6 +1,6 @@
 {
   "$schema": "https://raw.githubusercontent.com/AArnott/Nerdbank.GitVersioning/master/src/NerdBank.GitVersioning/version.schema.json",
-  "version": "2.12.18",
+  "version": "2.12.19",
   "versionHeightOffset": -1,
   "publicReleaseRefSpec": [
     "^refs/heads/main$",