From 69b1ab320d08decbdf2119fcb0300f7a346bcf8d Mon Sep 17 00:00:00 2001
From: benthecarman <benthecarman@live.com>
Date: Tue, 21 Feb 2023 11:26:06 -0600
Subject: [PATCH] Response to Tony's review

---
 Cargo.toml |  1 +
 src/lib.rs | 18 +++++-------------
 2 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/Cargo.toml b/Cargo.toml
index 48c8e5f..4f93bda 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -14,6 +14,7 @@ cfg-if = "0.1.2"
 # Need the queue feature
 worker = { git = "https://github.com/cloudflare/workers-rs", rev = "b4b9cd1f15feac412b6f9e9e9209458cd3b98430", features = ["queue"] }
 futures = "0.3.26"
+# Needed for serde serializaters
 nostr = { git = "https://github.com/benthecarman/nostr", rev = "6c238ac72651671bcfd6666ddcd731767d1643b8" }
 serde = { version = "^1.0", features = ["derive"] }
 
diff --git a/src/lib.rs b/src/lib.rs
index 9bdb363..bf367bd 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -191,27 +191,19 @@ pub async fn main(req: Request, env: Env, _ctx: Context) -> Result<Response> {
                                         {
                                             Ok(_) => {
                                                 console_log!("saved published note: {}", event.id);
-                                                let relay_msg =
-                                                    RelayMessage::new_ok(event.id, true, "");
-                                                server
-                                                    .send_with_str(&relay_msg.as_json())
-                                                    .expect("failed to send response");
                                             }
                                             Err(e) => {
                                                 console_log!(
                                                     "could not save published note: {} - {e:?}",
                                                     event.id
                                                 );
-                                                let relay_msg = RelayMessage::new_ok(
-                                                    event.id,
-                                                    false,
-                                                    "error: could not save published note",
-                                                );
-                                                server
-                                                    .send_with_str(&relay_msg.as_json())
-                                                    .expect("failed to send response");
                                             }
                                         }
+
+                                        let relay_msg = RelayMessage::new_ok(event.id, true, "");
+                                        server
+                                            .send_with_str(&relay_msg.as_json())
+                                            .expect("failed to send response");
                                     }
                                     _ => {
                                         console_log!("ignoring other nostr client message types");