From 9eeef7087492c20103a6514c46c851df6cfe2757 Mon Sep 17 00:00:00 2001
From: Haiping Chen <haiping008@gmail.com>
Date: Wed, 17 Jul 2024 07:05:16 -0500
Subject: [PATCH] not call fetched if exception errors.

---
 .../Drivers/PlaywrightDriver/PlaywrightInstance.cs | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/Plugins/BotSharp.Plugin.WebDriver/Drivers/PlaywrightDriver/PlaywrightInstance.cs b/src/Plugins/BotSharp.Plugin.WebDriver/Drivers/PlaywrightDriver/PlaywrightInstance.cs
index a0fae6f3..a1716929 100644
--- a/src/Plugins/BotSharp.Plugin.WebDriver/Drivers/PlaywrightDriver/PlaywrightInstance.cs
+++ b/src/Plugins/BotSharp.Plugin.WebDriver/Drivers/PlaywrightDriver/PlaywrightInstance.cs
@@ -115,16 +115,20 @@ public class PlaywrightInstance : IDisposable
                     JsonElement? json = null;
                     try
                     {
-                        json = await e.JsonAsync();
+                        if (e.Status == 200 && e.Ok)
+                        {
+                            json = await e.JsonAsync();
+                        }
+                        else
+                        {
+                            Serilog.Log.Warning($"Response status: {e.Status} {e.StatusText}, OK: {e.Ok}");
+                        }
+                        fetched(e.Url.ToLower(), JsonSerializer.Serialize(json));
                     }
                     catch(Exception ex)
                     {
                         Serilog.Log.Error(ex.ToString());
                     }
-                    finally
-                    {
-                        fetched(e.Url.ToLower(), JsonSerializer.Serialize(json ?? new JsonElement()));
-                    }
                 }
             };
         }