fix: normalize release commit timestamps
This commit is contained in:
@@ -132,8 +132,8 @@ internal class ImportDeveloperReleaseCommitsHandler(
|
|||||||
Subject = dto.Subject.Trim(),
|
Subject = dto.Subject.Trim(),
|
||||||
AuthorName = NormalizeOptional(dto.AuthorName),
|
AuthorName = NormalizeOptional(dto.AuthorName),
|
||||||
AuthorEmail = NormalizeOptional(dto.AuthorEmail),
|
AuthorEmail = NormalizeOptional(dto.AuthorEmail),
|
||||||
AuthoredAt = dto.AuthoredAt,
|
AuthoredAt = ToUtc(dto.AuthoredAt),
|
||||||
CommittedAt = dto.CommittedAt,
|
CommittedAt = ToUtc(dto.CommittedAt),
|
||||||
SourceBranch = NormalizeOptional(dto.SourceBranch),
|
SourceBranch = NormalizeOptional(dto.SourceBranch),
|
||||||
DeploymentLabel = NormalizeOptional(dto.DeploymentLabel),
|
DeploymentLabel = NormalizeOptional(dto.DeploymentLabel),
|
||||||
ExternalUrl = NormalizeOptional(dto.ExternalUrl),
|
ExternalUrl = NormalizeOptional(dto.ExternalUrl),
|
||||||
@@ -148,4 +148,9 @@ internal class ImportDeveloperReleaseCommitsHandler(
|
|||||||
string? normalized = value?.Trim();
|
string? normalized = value?.Trim();
|
||||||
return string.IsNullOrWhiteSpace(normalized) ? null : normalized;
|
return string.IsNullOrWhiteSpace(normalized) ? null : normalized;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private static DateTimeOffset? ToUtc(DateTimeOffset? value)
|
||||||
|
{
|
||||||
|
return value?.ToUniversalTime();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -209,8 +209,8 @@ internal sealed class ReleaseCommitRepositoryImportService(
|
|||||||
Subject = subject.Trim(),
|
Subject = subject.Trim(),
|
||||||
AuthorName = authorElement.HasValue ? NormalizeOptional(GetString(authorElement.Value, "name")) : null,
|
AuthorName = authorElement.HasValue ? NormalizeOptional(GetString(authorElement.Value, "name")) : null,
|
||||||
AuthorEmail = authorElement.HasValue ? NormalizeOptional(GetString(authorElement.Value, "email")) : null,
|
AuthorEmail = authorElement.HasValue ? NormalizeOptional(GetString(authorElement.Value, "email")) : null,
|
||||||
AuthoredAt = authorElement.HasValue ? GetDateTimeOffset(authorElement.Value, "date") : null,
|
AuthoredAt = authorElement.HasValue ? GetUtcDateTimeOffset(authorElement.Value, "date") : null,
|
||||||
CommittedAt = committerElement.HasValue ? GetDateTimeOffset(committerElement.Value, "date") : null,
|
CommittedAt = committerElement.HasValue ? GetUtcDateTimeOffset(committerElement.Value, "date") : null,
|
||||||
SourceBranch = NormalizeOptional(request.SourceBranch),
|
SourceBranch = NormalizeOptional(request.SourceBranch),
|
||||||
DeploymentLabel = NormalizeOptional(request.DeploymentLabel),
|
DeploymentLabel = NormalizeOptional(request.DeploymentLabel),
|
||||||
ExternalUrl = NormalizeOptional(GetString(commitElement, "html_url") ?? GetString(commitElement, "url")),
|
ExternalUrl = NormalizeOptional(GetString(commitElement, "html_url") ?? GetString(commitElement, "url")),
|
||||||
@@ -238,11 +238,11 @@ internal sealed class ReleaseCommitRepositoryImportService(
|
|||||||
: null;
|
: null;
|
||||||
}
|
}
|
||||||
|
|
||||||
private static DateTimeOffset? GetDateTimeOffset(JsonElement element, string propertyName)
|
private static DateTimeOffset? GetUtcDateTimeOffset(JsonElement element, string propertyName)
|
||||||
{
|
{
|
||||||
string? value = GetString(element, propertyName);
|
string? value = GetString(element, propertyName);
|
||||||
return DateTimeOffset.TryParse(value, CultureInfo.InvariantCulture, DateTimeStyles.AssumeUniversal, out DateTimeOffset result)
|
return DateTimeOffset.TryParse(value, CultureInfo.InvariantCulture, DateTimeStyles.AssumeUniversal, out DateTimeOffset result)
|
||||||
? result
|
? result.ToUniversalTime()
|
||||||
: null;
|
: null;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user