fix: allow parsing of commits with broken timestamp (#1322).

Instead, default to UTC just like Git.

Author: Byron

gix-object/tests/commit/from_bytes.rs

@@ -8,6 +8,33 @@ use crate::{
     fixture_name, linus_signature, signature,
 };
 
+#[test]
+fn invalid_timestsamp() {
+    let actor = gix_actor::SignatureRef {
+        name: b"Name".as_bstr(),
+        email: b"[email protected]".as_bstr(),
+        time: Time {
+            seconds: 1312735823,
+            offset: 0,
+            sign: Sign::Plus,
+        },
+    };
+    assert_eq!(
+        CommitRef::from_bytes(&fixture_name("commit", "invalid-timestamp.txt"))
+            .expect("auto-correct invalid timestamp by discarding it (time is still valid UTC)"),
+        CommitRef {
+            tree: b"7989dfb2ec2f41914611a22fb30bbc2b3849df9a".as_bstr(),
+            parents: [b"8845ae683e2688bc619baade49510c17e978518f".as_bstr()].into(),
+            author: actor,
+            committer: actor,
+            encoding: None,
+            message: b"edit changelog to mention about x_sendfile_header default change".as_bstr(),
+            extra_headers: vec![]
+        },
+        "the offset of the actor is null, leaving the UTC time"
+    );
+}
+
 #[test]
 fn unsigned() -> crate::Result {
     assert_eq!(

gix-object/tests/fixtures/commit/invalid-timestamp.txt

@@ -0,0 +1,6 @@
+tree 7989dfb2ec2f41914611a22fb30bbc2b3849df9a
+parent 8845ae683e2688bc619baade49510c17e978518f
+author Name <[email protected]> 1312735823 +051800
+committer Name <[email protected]> 1312735823 +051800
+
+edit changelog to mention about x_sendfile_header default change