Skip to content

Commit

Permalink
Cleanup rebase from release branch
Browse files Browse the repository at this point in the history
  • Loading branch information
webprofusion-chrisc committed Aug 15, 2024
1 parent 0b3acfc commit da58b0f
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 29 deletions.
2 changes: 1 addition & 1 deletion src/Certify.Client/CertifyServiceClient.cs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
using System;
using System;
using System.IO;
using System.Threading.Tasks;
using Certify.Models;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,29 +15,6 @@ namespace Certify.Management
{
public partial class CertifyManager
{
/// <summary>
/// Upgrade/migrate settings from previous version if applicable
/// </summary>
/// <returns></returns>
private async Task UpgradeSettings()
{
var systemVersion = Util.GetAppVersion().ToString();
var previousVersion = CoreAppSettings.Current.CurrentServiceVersion;

if (CoreAppSettings.Current.CurrentServiceVersion != systemVersion)
{
_tc?.TrackEvent("ServiceUpgrade", new Dictionary<string, string> {
{ "previousVersion", previousVersion },
{ "currentVersion", systemVersion }
});

// service has been updated, run any required migrations
await PerformServiceUpgrades();

CoreAppSettings.Current.CurrentServiceVersion = systemVersion;
SettingsManager.SaveAppSettings();
}
}

/// <summary>
/// Upgrade/migrate settings from previous version if applicable
Expand Down
2 changes: 0 additions & 2 deletions src/Certify.Models/Config/CertRequestConfig.cs
Original file line number Diff line number Diff line change
Expand Up @@ -343,8 +343,6 @@ internal List<string> GetCertificateDomains()

private static JsonSerializerOptions _defaultJsonSerializerOptions = new JsonSerializerOptions { PropertyNameCaseInsensitive = true };

private static JsonSerializerOptions _defaultJsonSerializerOptions = new JsonSerializerOptions { PropertyNameCaseInsensitive = true };

public List<CertIdentifierItem> GetCertificateIdentifiers()
{
var identifiers = new List<CertIdentifierItem>();
Expand Down
3 changes: 3 additions & 0 deletions src/Certify.Models/Util/EnvironmentUtil.cs
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
using System;
using System.Collections.Generic;
using System.IO;
using System.Runtime.InteropServices;
using System.Security.AccessControl;
using System.Security.Principal;

namespace Certify.Models
{
Expand Down
3 changes: 0 additions & 3 deletions src/Certify.Shared/Management/CertificateManager.cs
Original file line number Diff line number Diff line change
Expand Up @@ -752,9 +752,6 @@ private static string GetWindowsPrivateKeyLocation(string keyFileName)
// if EC/CNG key may be under /keys
machineKeyPath = Path.Combine(new string[] { appDataPath, "Microsoft", "Crypto", "Keys" });

// if EC/CNG key may be under /keys
machineKeyPath = Path.Combine(new string[] { appDataPath, "Microsoft", "Crypto", "Keys" });

fileList = Directory.GetFiles(machineKeyPath, keyFileName);

if (fileList.Any())
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
using System;
using System.Collections.Generic;
using System.Collections.ObjectModel;
using System.Linq;
Expand Down

0 comments on commit da58b0f

Please sign in to comment.