Make Send bounds explicit per API

[yukibtc]

Follow-up of ae1f2fc

Fixes #1399

[codecov]

Codecov Report

❌ Patch coverage is 53.93258% with 82 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
database/nostr-ndb/src/lib.rs	0.00%	17 Missing ⚠️
database/nostr-database/src/ext.rs	0.00%	14 Missing ⚠️
crates/nostr/src/event/builder.rs	23.52%	13 Missing ⚠️
sdk/src/events_tracker.rs	41.17%	10 Missing ⚠️
signer/nostr-browser-signer-proxy/src/lib.rs	0.00%	8 Missing ⚠️
crates/nostr/src/event/unsigned.rs	0.00%	4 Missing ⚠️
crates/nostr/src/nips/nip46.rs	0.00%	4 Missing ⚠️
signer/nostr-connect/src/client.rs	50.00%	4 Missing ⚠️
crates/nostr/src/key/public_key.rs	0.00%	3 Missing ⚠️
crates/nostr/src/key/mod.rs	75.00%	2 Missing ⚠️
... and 3 more

📢 Thoughts on this report? Let us know!

[reyakov]

it's fully work now. but the nostr-connect isn't fully migrate yet.

[yukibtc]

it's fully work now. but the nostr-connect isn't fully migrate yet.

What error do you get?

[reyakov]

it's fully work now. but the nostr-connect isn't fully migrate yet.

What error do you get?

I got this error:

rror: future cannot be sent between threads safely
   --> /Users/reya/.cargo/git/checkouts/nostr-619b808bb247a9ed/edf9ab4/signer/nostr-connect/src/client.rs:541:9
    |
541 |         Box::pin(async move { self._nip44_decrypt(*public_key, payload.to_string()).await })
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ future created by async block is not `Send`
    |
    = help: the trait `std::marker::Send` is not implemented for `dyn Future<Output = Result<Output<SubscriptionId>, Error>>`
note: future is not `Send` as this value is used across an await

[yukibtc]

@reyakov, it should be fixed now

[reyakov]

@reyakov, it should be fixed now

it's work fine now.