database: avoid to copy EventId in Event::decode

Signed-off-by: Yuki Kishimoto <[email protected]>

Author: yukibtc

CHANGELOG.md

@@ -38,6 +38,7 @@
 * pool: better checks before perform queries or send messages to relays ([Yuki Kishimoto])
 * pool: bump `async-wsocket` to `v0.6` ([Yuki Kishimoto])
 * database: not match event if `Filter::search` field is set ([Yuki Kishimoto])
+* database: avoid to copy `EventId` in `Event::decode` ([Yuki Kishimoto])
 * sdk: rename `Proxy` and `ProxyTarget` to `Connection` and `ConnectionTarget` ([Yuki Kishimoto])
 * sqlite: use `ValueRef` instead of owned one ([Yuki Kishimoto])
 * cli: improve `sync` command ([Yuki Kishimoto])

crates/nostr-database/src/flatbuffers/mod.rs

@@ -113,7 +113,7 @@ impl FlatBufferDecode for Event {
             .collect::<Result<Vec<Tag>, _>>()?;
 
         Ok(Self::new(
-            EventId::from_slice(&ev.id().ok_or(Error::NotFound)?.0)?,
+            EventId::owned(ev.id().ok_or(Error::NotFound)?.0),
             PublicKey::from_slice(&ev.pubkey().ok_or(Error::NotFound)?.0)?,
             Timestamp::from(ev.created_at()),
             Kind::from(ev.kind() as u16),