From c848c02237fe7fe98344707e60abc7c885f8190c Mon Sep 17 00:00:00 2001
From: Yuki Kishimoto <yukikishimoto@protonmail.com>
Date: Thu, 22 Feb 2024 13:46:37 +0100
Subject: [PATCH] signer: remove `Version` from `NostrSigner::nip44_encrypt`

---
 bindings/nostr-sdk-ffi/src/client/signer/mod.rs | 15 ++-------------
 bindings/nostr-sdk-js/src/client/signer/mod.rs  | 10 ++--------
 crates/nostr-sdk/src/client/mod.rs              |  4 +---
 crates/nostr-signer/src/lib.rs                  | 11 ++---------
 4 files changed, 7 insertions(+), 33 deletions(-)

diff --git a/bindings/nostr-sdk-ffi/src/client/signer/mod.rs b/bindings/nostr-sdk-ffi/src/client/signer/mod.rs
index 755d1f21e..b314ad42b 100644
--- a/bindings/nostr-sdk-ffi/src/client/signer/mod.rs
+++ b/bindings/nostr-sdk-ffi/src/client/signer/mod.rs
@@ -5,7 +5,6 @@
 use std::ops::Deref;
 use std::sync::Arc;
 
-use nostr_ffi::nips::nip44::Nip44Version;
 use nostr_ffi::{Event, EventBuilder, Keys, PublicKey, UnsignedEvent};
 use nostr_sdk::{block_on, signer};
 use uniffi::Object;
@@ -94,18 +93,8 @@ impl NostrSigner {
         })
     }
 
-    pub fn nip44_encrypt(
-        &self,
-        public_key: Arc<PublicKey>,
-        content: String,
-        version: Nip44Version,
-    ) -> Result<String> {
-        block_on(async move {
-            Ok(self
-                .inner
-                .nip44_encrypt(**public_key, content, version.into())
-                .await?)
-        })
+    pub fn nip44_encrypt(&self, public_key: Arc<PublicKey>, content: String) -> Result<String> {
+        block_on(async move { Ok(self.inner.nip44_encrypt(**public_key, content).await?) })
     }
 
     pub fn nip44_decrypt(&self, public_key: Arc<PublicKey>, content: String) -> Result<String> {
diff --git a/bindings/nostr-sdk-js/src/client/signer/mod.rs b/bindings/nostr-sdk-js/src/client/signer/mod.rs
index 20766301f..abde090d1 100644
--- a/bindings/nostr-sdk-js/src/client/signer/mod.rs
+++ b/bindings/nostr-sdk-js/src/client/signer/mod.rs
@@ -8,7 +8,6 @@ use nostr_js::error::{into_err, Result};
 use nostr_js::event::{JsEvent, JsEventBuilder, JsUnsignedEvent};
 use nostr_js::key::{JsKeys, JsPublicKey};
 use nostr_js::nips::nip07::JsNip07Signer;
-use nostr_js::nips::nip44::JsNIP44Version;
 use nostr_sdk::NostrSigner;
 use wasm_bindgen::prelude::*;
 
@@ -105,14 +104,9 @@ impl JsNostrSigner {
     }
 
     #[wasm_bindgen(js_name = nip44Encrypt)]
-    pub async fn nip44_encrypt(
-        &self,
-        public_key: &JsPublicKey,
-        content: String,
-        version: JsNIP44Version,
-    ) -> Result<String> {
+    pub async fn nip44_encrypt(&self, public_key: &JsPublicKey, content: String) -> Result<String> {
         self.inner
-            .nip44_encrypt(**public_key, content, version.into())
+            .nip44_encrypt(**public_key, content)
             .await
             .map_err(into_err)
     }
diff --git a/crates/nostr-sdk/src/client/mod.rs b/crates/nostr-sdk/src/client/mod.rs
index 8b30b0779..64090e016 100644
--- a/crates/nostr-sdk/src/client/mod.rs
+++ b/crates/nostr-sdk/src/client/mod.rs
@@ -1241,9 +1241,7 @@ impl Client {
         let rumor = rumor.to_unsigned_event(public_key);
 
         // Compose seal
-        let content: String = signer
-            .nip44_encrypt(receiver, rumor.as_json(), nip44::Version::default())
-            .await?;
+        let content: String = signer.nip44_encrypt(receiver, rumor.as_json()).await?;
         let seal: EventBuilder = EventBuilder::new(Kind::Seal, content, []);
         let seal: Event = self.sign_event_builder(seal).await?;
 
diff --git a/crates/nostr-signer/src/lib.rs b/crates/nostr-signer/src/lib.rs
index 863d934b5..82db01dd5 100644
--- a/crates/nostr-signer/src/lib.rs
+++ b/crates/nostr-signer/src/lib.rs
@@ -232,15 +232,8 @@ impl NostrSigner {
     }
 
     /// NIP44 encryption with [NostrSigner]
-    ///
-    /// Note: `Version` is ignored for NIP07!
     #[cfg(feature = "nip44")]
-    pub async fn nip44_encrypt<T>(
-        &self,
-        public_key: PublicKey,
-        content: T,
-        version: nip44::Version,
-    ) -> Result<String, Error>
+    pub async fn nip44_encrypt<T>(&self, public_key: PublicKey, content: T) -> Result<String, Error>
     where
         T: AsRef<str>,
     {
@@ -250,7 +243,7 @@ impl NostrSigner {
                 keys.secret_key()?,
                 &public_key,
                 content,
-                version,
+                nip44::Version::default(),
             )?),
             #[cfg(all(feature = "nip07", target_arch = "wasm32"))]
             Self::NIP07(signer) => Ok(signer.nip44_encrypt(public_key, content).await?),