diff --git a/crates/nu-command/src/formats/to/text.rs b/crates/nu-command/src/formats/to/text.rs index 858c5fce29..393de80ac3 100644 --- a/crates/nu-command/src/formats/to/text.rs +++ b/crates/nu-command/src/formats/to/text.rs @@ -1,3 +1,4 @@ +use chrono::Datelike; use chrono_humanize::HumanTime; use nu_engine::command_prelude::*; use nu_protocol::{format_duration, shell_error::io::IoError, ByteStream, PipelineMetadata}; @@ -167,7 +168,17 @@ fn local_into_string( Value::Filesize { val, .. } => val.to_string(), Value::Duration { val, .. } => format_duration(val), Value::Date { val, .. } => { - format!("{} ({})", val.to_rfc2822(), HumanTime::from(val)) + format!( + "{} ({})", + { + if val.year() >= 0 { + val.to_rfc2822() + } else { + val.to_rfc3339() + } + }, + HumanTime::from(val) + ) } Value::Range { val, .. } => val.to_string(), Value::String { val, .. } => val, diff --git a/crates/nu-command/src/strings/format/date.rs b/crates/nu-command/src/strings/format/date.rs index fa7911361c..ad668ca510 100644 --- a/crates/nu-command/src/strings/format/date.rs +++ b/crates/nu-command/src/strings/format/date.rs @@ -1,5 +1,5 @@ use crate::{generate_strftime_list, parse_date_from_string}; -use chrono::{DateTime, Locale, TimeZone}; +use chrono::{DateTime, Datelike, Locale, TimeZone}; use nu_engine::command_prelude::*; use nu_utils::locale::{get_system_locale_string, LOCALE_OVERRIDE_ENV_VAR}; @@ -228,11 +228,29 @@ fn format_helper( fn format_helper_rfc2822(value: Value, span: Span) -> Value { let val_span = value.span(); match value { - Value::Date { val, .. } => Value::string(val.to_rfc2822(), span), + Value::Date { val, .. } => Value::string( + { + if val.year() >= 0 { + val.to_rfc2822() + } else { + val.to_rfc3339() + } + }, + span, + ), Value::String { val, .. } => { let dt = parse_date_from_string(&val, val_span); match dt { - Ok(x) => Value::string(x.to_rfc2822(), span), + Ok(x) => Value::string( + { + if x.year() >= 0 { + x.to_rfc2822() + } else { + x.to_rfc3339() + } + }, + span, + ), Err(e) => e, } } diff --git a/crates/nu-command/tests/commands/into_datetime.rs b/crates/nu-command/tests/commands/into_datetime.rs index 83f36a7c26..8ad096ef0a 100644 --- a/crates/nu-command/tests/commands/into_datetime.rs +++ b/crates/nu-command/tests/commands/into_datetime.rs @@ -14,6 +14,16 @@ fn into_datetime_from_record() { assert_eq!(expected.out, actual.out); } +#[test] +fn into_datetime_from_record_very_old() { + let actual = nu!(r#"{year: -100, timezone: '+02:00'} | into datetime | into record"#); + let expected = nu!( + r#"{year: -100, month: 1, day: 1, hour: 0, minute: 0, second: 0, millisecond: 0, microsecond: 0, nanosecond: 0, timezone: '+02:00'}"# + ); + + assert_eq!(expected.out, actual.out); +} + #[test] fn into_datetime_from_record_defaults() { let actual = nu!(r#"{year: 2025, timezone: '+02:00'} | into datetime | into record"#); diff --git a/crates/nu-protocol/src/value/mod.rs b/crates/nu-protocol/src/value/mod.rs index 2d820b044a..0aac797c8d 100644 --- a/crates/nu-protocol/src/value/mod.rs +++ b/crates/nu-protocol/src/value/mod.rs @@ -4020,10 +4020,19 @@ fn operator_type_error( fn human_time_from_now(val: &DateTime) -> HumanTime { let now = Local::now().with_timezone(val.offset()); let delta = *val - now; - let delta_seconds = delta.num_nanoseconds().unwrap_or(0) as f64 / 1_000_000_000.0; - let delta_seconds_rounded = delta_seconds.round() as i64; - - HumanTime::from(Duration::seconds(delta_seconds_rounded)) + match delta.num_nanoseconds() { + Some(num_nanoseconds) => { + let delta_seconds = num_nanoseconds as f64 / 1_000_000_000.0; + let delta_seconds_rounded = delta_seconds.round() as i64; + HumanTime::from(Duration::seconds(delta_seconds_rounded)) + } + None => { + // Happens if the total number of nanoseconds exceeds what fits in an i64 + // Note: not using delta.num_days() because it results is wrong for years before ~936: a extra year is added + let delta_years = val.year() - now.year(); + HumanTime::from(Duration::days(delta_years as i64 * 365)) + } + } } #[cfg(test)]