Skip to content

Commit 0d9c696

Browse files
author
Lee Fine
committed
1 parent df0170a commit 0d9c696

6 files changed

Lines changed: 3 additions & 14 deletions

File tree

RemoteFile.UnitTests/ApplicationSettingsTests.cs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,16 +8,12 @@ public class ApplicationSettingsTests
88
[Fact]
99
public void FileTransferProtocol_WhenPopulatedWithValidValue_ReturnsValue()
1010
{
11-
var path = Path.Combine(Directory.GetCurrentDirectory(), "fixtures", "config", "valid", "config.json");
12-
ApplicationSettings.Initialize(path);
1311
Assert.Equal(ApplicationSettings.FileTransferProtocolEnum.SCP, ApplicationSettings.FileTransferProtocol);
1412
}
1513

1614
[Fact]
1715
public void FileTransferProtocol_WhenAllThreePopulated_DefaultsToBoth()
1816
{
19-
var path = Path.Combine(Directory.GetCurrentDirectory(), "fixtures", "config", "file_transfer_protocol_all_three", "config.json");
20-
ApplicationSettings.Initialize(path);
2117
Assert.Equal(ApplicationSettings.FileTransferProtocolEnum.Both, ApplicationSettings.FileTransferProtocol);
2218
}
2319
}

RemoteFile/ApplicationSettings.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
using Newtonsoft.Json;
1313
using Microsoft.Extensions.Logging;
1414
using Keyfactor.Logging;
15+
using System.Reflection;
1516

1617

1718
namespace Keyfactor.Extensions.Orchestrator.RemoteFile
@@ -88,13 +89,13 @@ public static FileTransferProtocolEnum FileTransferProtocol
8889
}
8990
}
9091

91-
public static void Initialize(string configLocation)
92+
static ApplicationSettings()
9293
{
9394
ILogger logger = LogHandler.GetClassLogger<ApplicationSettings>();
9495
logger.MethodEntry(LogLevel.Debug);
9596

9697
configuration = new Dictionary<string, string>();
97-
configLocation = $"{Path.GetDirectoryName(configLocation)}{Path.DirectorySeparatorChar}config.json";
98+
string configLocation = $"{Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location)}{Path.DirectorySeparatorChar}config.json";
9899
string configContents = string.Empty;
99100

100101
if (!File.Exists(configLocation))

RemoteFile/Discovery.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,6 @@ public JobResult ProcessJob(DiscoveryJobConfiguration config, SubmitDiscoveryUpd
5656
string userName = PAMUtilities.ResolvePAMField(_resolver, logger, "Server User Name", config.ServerUsername);
5757
string userPassword = PAMUtilities.ResolvePAMField(_resolver, logger, "Server Password", config.ServerPassword);
5858

59-
ApplicationSettings.Initialize(this.GetType().Assembly.Location);
60-
6159
certificateStore = new RemoteCertificateStore(config.ClientMachine, userName, userPassword, directoriesToSearch[0].Substring(0, 1) == "/" ? RemoteCertificateStore.ServerTypeEnum.Linux : RemoteCertificateStore.ServerTypeEnum.Windows, ApplicationSettings.SSHPort);
6260
certificateStore.Initialize(ApplicationSettings.DefaultSudoImpersonatedUser);
6361

RemoteFile/InventoryBase.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ public JobResult ProcessJob(InventoryJobConfiguration config, SubmitInventoryUpd
3535

3636
try
3737
{
38-
ApplicationSettings.Initialize(this.GetType().Assembly.Location);
39-
4038
SetJobProperties(config, config.CertificateStoreDetails, logger);
4139

4240
certificateStore = new RemoteCertificateStore(config.CertificateStoreDetails.ClientMachine, UserName, UserPassword, config.CertificateStoreDetails.StorePath, StorePassword, FileTransferProtocol, SSHPort, IncludePortInSPN);

RemoteFile/ManagementBase.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@ public JobResult ProcessJob(ManagementJobConfiguration config)
3333

3434
try
3535
{
36-
ApplicationSettings.Initialize(this.GetType().Assembly.Location);
37-
3836
SetJobProperties(config, config.CertificateStoreDetails, logger);
3937

4038
certificateStore = new RemoteCertificateStore(config.CertificateStoreDetails.ClientMachine, UserName, UserPassword, config.CertificateStoreDetails.StorePath, StorePassword, FileTransferProtocol, SSHPort, IncludePortInSPN);

RemoteFile/ReenrollmentBase.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ public JobResult ProcessJobToDo(ReenrollmentJobConfiguration config, SubmitReenr
5050

5151
try
5252
{
53-
ApplicationSettings.Initialize(this.GetType().Assembly.Location);
54-
5553
SetJobProperties(config, config.CertificateStoreDetails, logger);
5654

5755
string alias = "abcd";

0 commit comments

Comments
 (0)