Realign property name casing in DeviceClientDto. Move update check after reconnection, so these can be properly cased later.

This commit is contained in:
Jared Goodwin 2023-06-24 10:06:00 -07:00
parent 097e6a45cf
commit f1e83df87a
8 changed files with 29 additions and 29 deletions

View File

@ -109,7 +109,6 @@ namespace Remotely.Agent.Services
_logger.LogInformation("Connected to server.");
// TODO: Move CPU sampler to background service.
var device = await _deviceInfoService.CreateDevice(_connectionInfo.DeviceID, _connectionInfo.OrganizationID);
var result = await _hubConnection.InvokeAsync<bool>("DeviceCameOnline", device);
@ -199,6 +198,8 @@ namespace Remotely.Agent.Services
private async Task HubConnection_Reconnected(string arg)
{
_logger.LogInformation("Reconnected to server.");
await _updater.CheckForUpdates();
var device = await _deviceInfoService.CreateDevice(_connectionInfo.DeviceID, _connectionInfo.OrganizationID);
if (!await _hubConnection.InvokeAsync<bool>("DeviceCameOnline", device))
@ -212,9 +213,8 @@ namespace Remotely.Agent.Services
await Connect();
return;
}
await _updater.CheckForUpdates();
}
private void RegisterMessageHandlers()
{

View File

@ -27,16 +27,16 @@ namespace Remotely.Agent.Services
return new DeviceClientDto()
{
Id = deviceID,
ID = deviceID,
DeviceName = Environment.MachineName,
Platform = EnvironmentHelper.Platform.ToString(),
ProcessorCount = Environment.ProcessorCount,
OsArchitecture = RuntimeInformation.OSArchitecture,
OsDescription = RuntimeInformation.OSDescription,
OSArchitecture = RuntimeInformation.OSArchitecture,
OSDescription = RuntimeInformation.OSDescription,
Is64Bit = Environment.Is64BitOperatingSystem,
IsOnline = true,
MacAddresses = GetMacAddresses().ToArray(),
OrganizationId = orgID,
OrganizationID = orgID,
AgentVersion = AppVersionHelper.GetAppVersion()
};
}

View File

@ -92,7 +92,7 @@ namespace Remotely.Server.Hubs
{
try
{
if (CheckForDeviceBan(device.Id, device.DeviceName))
if (CheckForDeviceBan(device.ID, device.DeviceName))
{
return false;
}
@ -147,7 +147,7 @@ namespace Remotely.Server.Hubs
public async Task DeviceHeartbeat(DeviceClientDto device)
{
if (CheckForDeviceBan(device.Id, device.DeviceName))
if (CheckForDeviceBan(device.ID, device.DeviceName))
{
return;
}

View File

@ -360,14 +360,14 @@ namespace Remotely.Server.Services
{
using var dbContext = _appDbFactory.GetContext();
var device = await dbContext.Devices.FindAsync(deviceDto.Id);
var device = await dbContext.Devices.FindAsync(deviceDto.ID);
if (device is null)
{
device = new Device
{
OrganizationID = deviceDto.OrganizationId,
ID = deviceDto.Id,
OrganizationID = deviceDto.OrganizationID,
ID = deviceDto.ID,
};
await dbContext.Devices.AddAsync(device);
}
@ -380,8 +380,8 @@ namespace Remotely.Server.Services
device.UsedStorage = deviceDto.UsedStorage;
device.Is64Bit = deviceDto.Is64Bit;
device.IsOnline = true;
device.OSArchitecture = deviceDto.OsArchitecture;
device.OSDescription = deviceDto.OsDescription;
device.OSArchitecture = deviceDto.OSArchitecture;
device.OSDescription = deviceDto.OSDescription;
device.Platform = deviceDto.Platform;
device.ProcessorCount = deviceDto.ProcessorCount;
device.PublicIP = deviceDto.PublicIP;

View File

@ -29,7 +29,7 @@ namespace Remotely.Shared.Dtos
public List<Drive> Drives { get; set; } = new();
[DataMember]
public string Id { get; set; } = string.Empty;
public string ID { get; set; } = string.Empty;
[DataMember]
public bool Is64Bit { get; set; }
@ -41,13 +41,13 @@ namespace Remotely.Shared.Dtos
public string[] MacAddresses { get; set; } = Array.Empty<string>();
[DataMember]
public string OrganizationId { get; set; } = string.Empty;
public string OrganizationID { get; set; } = string.Empty;
[DataMember]
public Architecture OsArchitecture { get; set; }
public Architecture OSArchitecture { get; set; }
[DataMember]
public string OsDescription { get; set; } = string.Empty;
public string OSDescription { get; set; } = string.Empty;
[DataMember]
public string Platform { get; set; } = string.Empty;

View File

@ -116,7 +116,7 @@ internal class Program
{
try
{
var currentInfo = await _deviceInfo.CreateDevice(device.Id, _organizationId);
var currentInfo = await _deviceInfo.CreateDevice(device.ID, _organizationId);
currentInfo.DeviceName = device.DeviceName;
await hubConnection.SendAsync("DeviceHeartbeat", currentInfo);
}

View File

@ -38,8 +38,8 @@ namespace Remotely.Tests
var newDevice = new DeviceClientDto()
{
Id = _newDeviceID,
OrganizationId = _testData.Org1Id,
ID = _newDeviceID,
OrganizationID = _testData.Org1Id,
DeviceName = Environment.MachineName,
Is64Bit = Environment.Is64BitOperatingSystem
};

View File

@ -116,15 +116,15 @@ namespace Remotely.Tests
var device1 = new DeviceClientDto()
{
Id = "Org1Device1",
ID = "Org1Device1",
DeviceName = "Org1Device1Name",
OrganizationId = Org1Id
OrganizationID = Org1Id
};
var device2 = new DeviceClientDto()
{
Id = "Org1Device2",
ID = "Org1Device2",
DeviceName = "Org1Device2Name",
OrganizationId = Org1Id
OrganizationID = Org1Id
};
Org1Device1 = (await dataService.AddOrUpdateDevice(device1)).Value;
Org1Device2 = (await dataService.AddOrUpdateDevice(device2)).Value;
@ -150,15 +150,15 @@ namespace Remotely.Tests
var device3 = new DeviceClientDto()
{
Id = "Org2Device1",
ID = "Org2Device1",
DeviceName = "Org2Device1Name",
OrganizationId = Org2Id
OrganizationID = Org2Id
};
var device4 = new DeviceClientDto()
{
Id = "Org2Device2",
ID = "Org2Device2",
DeviceName = "Org2Device2Name",
OrganizationId = Org2Id
OrganizationID = Org2Id
};
Org2Device1 = (await dataService.AddOrUpdateDevice(device3)).Value;
Org2Device2 = (await dataService.AddOrUpdateDevice(device4)).Value;