chore(Cargo): bump rust-nostr to version 0.44

Author: jgmontoya

Cargo.lock

@@ -25,7 +25,7 @@ openmls_basic_credential = { git = "https://github.com/openmls/openmls", rev = "
 openmls_rust_crypto = { git = "https://github.com/openmls/openmls", rev = "b90ca23b1238d9e189142d06234527b2d344d748", default-features = false }
 
 # Nostr
-nostr = { version = "0.43", default-features = false }
+nostr = { version = "0.44", default-features = false }
 
 # Serialization
 serde = { version = "1.0", default-features = false }

Cargo.toml

@@ -23,6 +23,10 @@
 
 ## Unreleased
 
+### Changed
+
+- Upgraded `nostr` dependency from 0.43 to 0.44, replacing deprecated `Timestamp::as_u64()` calls with `Timestamp::as_secs()`
+
 ### Added
 
 - Configurable `out_of_order_tolerance` and `maximum_forward_distance` in `MdkConfig` for MLS sender ratchet settings. Default `out_of_order_tolerance` increased from 5 to 100 for better handling of out-of-order message delivery on Nostr relays. ([`#155`](https://github.com/marmot-protocol/mdk/pull/155))

crates/mdk-core/CHANGELOG.md

@@ -971,7 +971,7 @@ where
             &group_id,
             current_epoch,
             &event.id,
-            event.created_at.as_u64(),
+            event.created_at.as_secs(),
         ) {
             tracing::warn!(
                 target: "mdk_core::messages::process_commit_message_for_group",
@@ -1155,24 +1155,24 @@ where
         let now = Timestamp::now();
 
         // Reject events from the future (allow configurable clock skew)
-        if event.created_at.as_u64()
+        if event.created_at.as_secs()
             > now
-                .as_u64()
+                .as_secs()
                 .saturating_add(self.config.max_future_skew_secs)
         {
             return Err(Error::InvalidTimestamp(format!(
                 "event timestamp {} is too far in the future (current time: {})",
-                event.created_at.as_u64(),
-                now.as_u64()
+                event.created_at.as_secs(),
+                now.as_secs()
             )));
         }
 
         // Reject events that are too old (configurable via MdkConfig)
-        let min_timestamp = now.as_u64().saturating_sub(self.config.max_event_age_secs);
-        if event.created_at.as_u64() < min_timestamp {
+        let min_timestamp = now.as_secs().saturating_sub(self.config.max_event_age_secs);
+        if event.created_at.as_secs() < min_timestamp {
             return Err(Error::InvalidTimestamp(format!(
                 "event timestamp {} is too old (minimum acceptable: {})",
-                event.created_at.as_u64(),
+                event.created_at.as_secs(),
                 min_timestamp
             )));
         }
@@ -1401,7 +1401,7 @@ where
                         &group.mls_group_id,
                         mls_group.epoch().as_u64(),
                         &event.id,
-                        event.created_at.as_u64(),
+                        event.created_at.as_secs(),
                     )
                     .is_err()
                 {
@@ -1630,7 +1630,7 @@ where
                     self.storage(),
                     &group.mls_group_id,
                     msg_epoch,
-                    event.created_at.as_u64(),
+                    event.created_at.as_secs(),
                     &event.id,
                 );
 
@@ -3165,9 +3165,9 @@ mod tests {
         );
 
         // Allow for some clock skew, but message shouldn't be more than a day old
-        let one_day_ago = now.as_u64().saturating_sub(86400);
+        let one_day_ago = now.as_secs().saturating_sub(86400);
         assert!(
-            message_event.created_at.as_u64() > one_day_ago,
+            message_event.created_at.as_secs() > one_day_ago,
             "Message timestamp should be recent"
         );
     }
@@ -6562,7 +6562,7 @@ mod tests {
             .expect("Group should exist");
 
         // Set timestamp to far future (1 hour ahead, beyond 5 minute skew allowance)
-        let future_time = nostr::Timestamp::now().as_u64() + 3600;
+        let future_time = nostr::Timestamp::now().as_secs() + 3600;
 
         // Create an event with future timestamp
         let message_event = EventBuilder::new(Kind::MlsGroupMessage, "test content")
@@ -6597,7 +6597,7 @@ mod tests {
             .expect("Group should exist");
 
         // Set timestamp to 46 days ago (beyond 45 day limit)
-        let old_time = nostr::Timestamp::now().as_u64().saturating_sub(46 * 86400);
+        let old_time = nostr::Timestamp::now().as_secs().saturating_sub(46 * 86400);
 
         // Create an event with old timestamp
         let message_event = EventBuilder::new(Kind::MlsGroupMessage, "test content")

crates/mdk-core/src/messages.rs

@@ -233,7 +233,7 @@ impl RaceConditionSimulator {
 
     /// Get a timestamp offset from the base by the specified number of seconds
     pub fn timestamp_offset(&self, offset_seconds: i64) -> nostr::Timestamp {
-        let new_timestamp = (self.base_timestamp.as_u64() as i64 + offset_seconds).max(0) as u64;
+        let new_timestamp = (self.base_timestamp.as_secs() as i64 + offset_seconds).max(0) as u64;
         nostr::Timestamp::from(new_timestamp)
     }
 }

crates/mdk-core/src/test_util.rs

@@ -29,6 +29,7 @@
 
 ### Changed
 
+- Upgraded `nostr` dependency from 0.43 to 0.44, replacing deprecated `Timestamp::as_u64()` calls with `Timestamp::as_secs()`
 - **Persistent Snapshots**: Implemented snapshot support by copying group-specific rows to a dedicated snapshot table. `create_group_snapshot`, `rollback_group_to_snapshot`, and `release_group_snapshot` persist across app restarts. ([#152](https://github.com/marmot-protocol/mdk/pull/152))
 - **Unified Storage Architecture**: `MdkSqliteStorage` now directly implements OpenMLS's `StorageProvider<1>` trait instead of wrapping `openmls_sqlite_storage`. This enables atomic transactions across MLS and MDK state, which is required for proper commit race resolution per MIP-03. ([#148](https://github.com/marmot-protocol/mdk/pull/148))
   - Removed `openmls_sqlite_storage` dependency

crates/mdk-sqlite-storage/CHANGELOG.md

@@ -113,7 +113,7 @@ impl GroupStorage for MdkSqliteStorage {
 
         let last_message_id: Option<&[u8; 32]> =
             group.last_message_id.as_ref().map(|id| id.as_bytes());
-        let last_message_at: Option<u64> = group.last_message_at.as_ref().map(|ts| ts.as_u64());
+        let last_message_at: Option<u64> = group.last_message_at.as_ref().map(|ts| ts.as_secs());
 
         self.with_connection(|conn| {
             conn.execute(

crates/mdk-sqlite-storage/src/groups.rs

@@ -65,7 +65,7 @@ impl MessageStorage for MdkSqliteStorage {
                     message.pubkey.as_bytes(),
                     message.kind.as_u16(),
                     message.mls_group_id.as_slice(),
-                    message.created_at.as_u64(),
+                    message.created_at.as_secs(),
                     &message.content,
                     &tags_json,
                     &event_json,
@@ -123,7 +123,7 @@ impl MessageStorage for MdkSqliteStorage {
                 params![
                     &processed_message.wrapper_event_id.to_bytes(),
                     &message_event_id,
-                    &processed_message.processed_at.as_u64(),
+                    &processed_message.processed_at.as_secs(),
                     &processed_message.epoch,
                     &mls_group_id,
                     &processed_message.state.to_string(),

crates/mdk-sqlite-storage/src/messages.rs

@@ -171,7 +171,7 @@ impl WelcomeStorage for MdkSqliteStorage {
                 params![
                     processed_welcome.wrapper_event_id.as_bytes(),
                     welcome_event_id,
-                    processed_welcome.processed_at.as_u64(),
+                    processed_welcome.processed_at.as_secs(),
                     processed_welcome.state.as_str(),
                     &processed_welcome.failure_reason
                 ],

crates/mdk-sqlite-storage/src/welcomes.rs

@@ -28,6 +28,10 @@
 ### Breaking changes
 
 - **Security (Audit Issue M)**: Changed `get_message()` to require both `mls_group_id` and `event_id` parameters. This prevents messages from different groups from overwriting each other by scoping lookups to a specific group. ([#124](https://github.com/marmot-protocol/mdk/pull/124))
+
+### Changed
+
+- Upgraded `nostr` dependency from 0.43 to 0.44, replacing deprecated `Timestamp::as_u64()` calls with `Timestamp::as_secs()`
 - Changed `get_messages()` to accept optional `limit` and `offset` parameters for pagination control. Existing calls must be updated to pass `None, None` for default behavior (limit: 1000, offset: 0), or specify values for custom pagination. ([#111](https://github.com/marmot-protocol/mdk/pull/111))
 - Changed `get_pending_welcomes()` to accept optional `limit` and `offset` parameters for pagination control. Existing calls must be updated to pass `None, None` for default behavior (limit: 1000, offset: 0), or specify values for custom pagination. ([#119](https://github.com/marmot-protocol/mdk/pull/119))
 - Changed `new_mdk()`, `new_mdk_with_key()`, and `new_mdk_unencrypted()` to accept an optional `MdkConfig` parameter for customizing MDK behavior. Existing calls must be updated to pass `None` for default behavior. ([`#155`](https://github.com/marmot-protocol/mdk/pull/155))