OK message for ephemeral events

[kumulynja]

It seems like an OK message is only send for events that are persisted, not for ephemeral events.
This is concluded from testing with nip47 request and response events and from seeing the following issue for nip42 that also uses ephemeral events: #213

As NIP01 describes, all events should be acknowledged or denied by an OK message, so that also includes ephemeral events, as other relays do.

OK messages MUST be sent in response to EVENT messages received from clients, they must have the 3rd parameter set to true when an event has been accepted by the relay, false otherwise.

[Sjors]

Ah this is probably why noStrudel thinks my relay never posts my stuff, even though it does.

[TheAwiteb]

@scsibug I am interested in fixing this issue, can you point me?

[v0l]

nostr-rs-relay/src/db.rs

Lines 395 to 406 in a6b4862

	if event.is_ephemeral() {
	bcast_tx.send(event.clone()).ok();
	debug!(
	"published ephemeral event: {:?} from: {:?} in: {:?}",
	event.get_event_id_prefix(),
	event.get_author_prefix(),
	start.elapsed()
	);
	event_write = true;
	// send OK message
	notice_tx.try_send(Notice::saved(event.id)).ok();

[TheAwiteb]

nostr-rs-relay/src/db.rs

Lines 395 to 406 in a6b4862

 if event.is_ephemeral() { 
     bcast_tx.send(event.clone()).ok(); 
     debug!( 
         "published ephemeral event: {:?} from: {:?} in: {:?}", 
         event.get_event_id_prefix(), 
         event.get_author_prefix(), 
         start.elapsed() 
     ); 
     event_write = true; 
  
     // send OK message 
     notice_tx.try_send(Notice::saved(event.id)).ok();

So it sends it, why is this issue faced us?

[v0l]

So it sends it, why is this issue faced us?

Somebody needs to debug this