Merge pull request #933 from visagang/features/vguruparan
Update timeout for web driver logic to fetch from appSettings
This commit is contained in:
commit
b9e9d5865f
|
|
@ -4,4 +4,6 @@ public class WebBrowsingSettings
|
|||
{
|
||||
public string Driver { get; set; } = "Playwright";
|
||||
public bool Headless { get; set; }
|
||||
// Default timeout in milliseconds
|
||||
public float DefaultTimeout { get; set; } = 30000;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,3 +1,5 @@
|
|||
using BotSharp.Abstraction.Browsing.Settings;
|
||||
|
||||
namespace BotSharp.Plugin.WebDriver.Functions;
|
||||
|
||||
public class GoToPageFn : IFunctionCallback
|
||||
|
|
@ -25,7 +27,7 @@ public class GoToPageFn : IFunctionCallback
|
|||
var url = webDriverService.ReplaceToken(args.Url);
|
||||
|
||||
url = url.Replace("https://https://", "https://");
|
||||
|
||||
var _webDriver = _services.GetRequiredService<WebBrowsingSettings>();
|
||||
var result = await _browser.GoToPage(new MessageInfo
|
||||
{
|
||||
AgentId = message.CurrentAgentId,
|
||||
|
|
@ -33,7 +35,8 @@ public class GoToPageFn : IFunctionCallback
|
|||
MessageId = message.MessageId
|
||||
}, new PageActionArgs
|
||||
{
|
||||
Url = url
|
||||
Url = url,
|
||||
Timeout = _webDriver.DefaultTimeout
|
||||
});
|
||||
message.Content = result.IsSuccess ? $"Page {url} is open." : $"Page {url} open failed. {result.Message}";
|
||||
|
||||
|
|
|
|||
|
|
@ -26,7 +26,7 @@ public class OpenBrowserFn : IFunctionCallback
|
|||
|
||||
var webDriverService = _services.GetRequiredService<WebDriverService>();
|
||||
var url = webDriverService.ReplaceToken(args.Url);
|
||||
|
||||
var _webDriver = _services.GetRequiredService<WebBrowsingSettings>();
|
||||
url = url.Replace("https://https://", "https://");
|
||||
var msgInfo = new MessageInfo
|
||||
{
|
||||
|
|
@ -40,7 +40,8 @@ public class OpenBrowserFn : IFunctionCallback
|
|||
});
|
||||
result = await _browser.GoToPage(msgInfo, new PageActionArgs
|
||||
{
|
||||
Url = url
|
||||
Url = url,
|
||||
Timeout = _webDriver.DefaultTimeout
|
||||
});
|
||||
|
||||
if (result.IsSuccess)
|
||||
|
|
|
|||
|
|
@ -1,3 +1,5 @@
|
|||
using BotSharp.Abstraction.Browsing.Settings;
|
||||
|
||||
namespace BotSharp.Plugin.WebDriver.UtilFunctions;
|
||||
|
||||
public class UtilWebGoToPageFn : IFunctionCallback
|
||||
|
|
@ -21,6 +23,8 @@ public class UtilWebGoToPageFn : IFunctionCallback
|
|||
{
|
||||
PropertyNamingPolicy = JsonNamingPolicy.CamelCase
|
||||
});
|
||||
var _webDriver = _services.GetRequiredService<WebBrowsingSettings>();
|
||||
args.Timeout = _webDriver.DefaultTimeout;
|
||||
args.WaitForNetworkIdle = false;
|
||||
args.WaitTime = 5;
|
||||
args.OpenNewTab = true;
|
||||
|
|
|
|||
Loading…
Reference in a new issue