Skip to content
This repository was archived by the owner on Dec 24, 2022. It is now read-only.

Commit 39e36c2

Browse files
committed
Fix MaxDate.ToUnixTime() Issue
1 parent 2f6869a commit 39e36c2

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

src/ServiceStack.Text/DateTimeExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ private static TimeSpan ToDateTimeSinceUnixEpoch(this DateTime dateTime)
6262
var dtUtc = dateTime;
6363
if (dateTime.Kind != DateTimeKind.Utc)
6464
{
65-
dtUtc = dateTime.Kind == DateTimeKind.Unspecified && dateTime > DateTime.MinValue
65+
dtUtc = dateTime.Kind == DateTimeKind.Unspecified && dateTime > DateTime.MinValue && dateTime < DateTime.MaxValue
6666
? DateTime.SpecifyKind(dateTime.Subtract(DateTimeSerializer.LocalTimeZone.GetUtcOffset(dateTime)), DateTimeKind.Utc)
6767
: dateTime.ToStableUniversalTime();
6868
}

tests/ServiceStack.Text.Tests/Utils/DateTimeSerializerTests.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -297,6 +297,13 @@ public void Test_MS_Dates(int whichDate)
297297

298298
AssertDatesAreEqual(fromStr, dateTime);
299299
}
300+
301+
[Test]
302+
public void Can_serialize_MaxDateTime()
303+
{
304+
var maxDate = DateTime.MaxValue.ToUnixTime();
305+
var minDate = DateTime.MinValue.ToUnixTime();
306+
}
300307
}
301308

302309
public class DateModel

0 commit comments

Comments
 (0)