Skip to content

Commit 69a826a

Browse files
authored
[Feature/Enabling_Gecko_Driver_Host_Default_In_Settings] Add additional property to set geckoService.Host ::1 +semver: minor (#218)
* [Feature/Enabling_Gecko_Driver_Host_Default_In_Settings] Add additional property to set geckoService.Host ::1 * [Feature/Enabling_Gecko_Driver_Host_Default_In_Settings] Refactor code * [Feature/Enabling_Gecko_Driver_Host_Default_In_Settings] Refactor code
1 parent 5b371e9 commit 69a826a

File tree

3 files changed

+5
-1
lines changed

3 files changed

+5
-1
lines changed

Aquality.Selenium/src/Aquality.Selenium/Browsers/LocalBrowserFactory.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ namespace Aquality.Selenium.Browsers
2424
public class LocalBrowserFactory : BrowserFactory
2525
{
2626
private static readonly object WebDriverDownloadingLock = new object();
27+
private const string HostAddressDefault = "::1";
2728

2829
public LocalBrowserFactory(IActionRetrier actionRetrier, IBrowserProfile browserProfile, ITimeoutConfiguration timeoutConfiguration, ILocalizedLogger localizedLogger)
2930
: base(actionRetrier, browserProfile, timeoutConfiguration, localizedLogger)
@@ -49,7 +50,7 @@ protected override WebDriver Driver
4950
case BrowserName.Firefox:
5051
SetUpDriver(new FirefoxConfig(), driverSettings);
5152
var geckoService = FirefoxDriverService.CreateDefaultService();
52-
geckoService.Host = "::1";
53+
geckoService.Host = ((FirefoxSettings)driverSettings).IsGeckoServiceHostDefaultEnabled ? HostAddressDefault : geckoService.Host;
5354
driver = GetDriver<FirefoxDriver>(geckoService, (FirefoxOptions)driverSettings.DriverOptions, commandTimeout);
5455
break;
5556
case BrowserName.IExplorer:

Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/FirefoxSettings.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ public FirefoxSettings(ISettingsFile settingsFile) : base(settingsFile)
2323

2424
protected override BrowserName BrowserName => BrowserName.Firefox;
2525

26+
public bool IsGeckoServiceHostDefaultEnabled => SettingsFile.GetValueOrDefault($"{DriverSettingsPath}.isGeckoServiceHostDefaultEnabled", false);
27+
2628
protected override IDictionary<string, Action<DriverOptions, object>> KnownCapabilitySetters => new Dictionary<string, Action<DriverOptions, object>>
2729
{
2830
{ "binary", (options, value) => ((FirefoxOptions) options).BrowserExecutableLocation = value.ToString() },

Aquality.Selenium/src/Aquality.Selenium/Resources/settings.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
},
2626
"firefox": {
2727
"webDriverVersion": "Latest",
28+
"isGeckoServiceHostDefaultEnabled": false,
2829
"capabilities": {
2930
"enableVNC": true,
3031
"unhandledPromptBehavior": "default"

0 commit comments

Comments
 (0)