feat: event subscriptions

Author: Frando

iroh-willow/src/engine/actor.rs

@@ -22,8 +22,7 @@ use crate::{
     },
     session::{intents::Intent, run_session, Error, EventSender, SessionHandle},
     store::{
-        entry::EntryOrigin,
-        traits::{EntryReader, SecretStorage, Storage},
+        traits::{EntryOrigin, EntryReader, EntryStorage, SecretStorage, Storage},
         Store,
     },
 };
@@ -445,7 +444,7 @@ impl<S: Storage> Actor<S> {
                 origin,
                 reply,
             } => {
-                let res = self.store.entries().ingest(&authorised_entry, origin);
+                let res = self.store.entries().ingest_entry(&authorised_entry, origin);
                 send_reply(reply, res)
             }
             Input::InsertEntry { entry, auth, reply } => {

iroh-willow/src/proto/grouping.rs

@@ -37,6 +37,7 @@ pub type AreaOfInterest = willow_data_model::grouping::AreaOfInterest<
 >;
 
 /// Extension methods for [`AreaOfInterest`].
+// TODO: Upstream to willow-rs as methods on [`AreaOfInterest].
 pub trait AreaOfInterestExt {
     /// Creates a new area of interest with the specified area and no other limits.
     fn with_area(area: Area) -> AreaOfInterest;
@@ -53,6 +54,7 @@ impl AreaOfInterestExt for AreaOfInterest {
 }
 
 /// Extension methods for [`Area`].
+// TODO: Upstream to willow-rs as methods on [`Area`].
 pub trait AreaExt {
     /// Returns `true` if the area contains `point`.
     fn includes_point(&self, point: &Point) -> bool;
@@ -93,6 +95,7 @@ impl AreaExt for Area {
 /// A single point in the 3D range space.
 ///
 /// I.e. an entry.
+// TODO: Upstream to willow-rs.
 #[derive(Debug, Clone, Hash, Eq, PartialEq, Serialize, Deserialize)]
 pub struct Point {
     #[serde(with = "data_model::serde_encoding::path")]

iroh-willow/src/session/data.rs

@@ -1,5 +1,4 @@
 use futures_lite::StreamExt;
-use tokio::sync::broadcast;
 
 use crate::{
     proto::{
@@ -8,8 +7,7 @@ use crate::{
     },
     session::{channels::ChannelSenders, static_tokens::StaticTokens, Error, SessionId},
     store::{
-        entry::{EntryChannel, EntryOrigin},
-        traits::Storage,
+        traits::{EntryOrigin, EntryStorage, Storage, StoreEvent, SubscribeParams},
         Store,
     },
     util::stream::CancelableReceiver,
@@ -51,29 +49,36 @@ impl<S: Storage> DataSender<S> {
         }
     }
     pub async fn run(mut self) -> Result<(), Error> {
-        let mut entry_stream = self.store.entries().subscribe(self.session_id);
+        let mut entry_stream = futures_concurrency::stream::StreamGroup::new();
         loop {
             tokio::select! {
                 input = self.inbox.next() => {
                     let Some(input) = input else {
                         break;
                     };
                     let Input::AoiIntersection(intersection) = input;
-                    self.store.entries().watch_area(
-                        self.session_id,
-                        intersection.namespace,
-                        intersection.intersection.area.clone(),
-                    );
+                    let params = SubscribeParams::default().ingest_only().ignore_remote(self.session_id);
+                    // TODO: We could start at the progress id at the beginning of the session.
+                    let stream = self
+                        .store
+                        .entries()
+                        .subscribe_area(
+                            intersection.namespace,
+                            intersection.intersection.area.clone(),
+                            params,
+                        )
+                        .filter_map(|event| match event {
+                            StoreEvent::Ingested(_id, entry, _origin) => Some(entry),
+                            // We get only Ingested events because we set ingest_only() param above.
+                            _ => unreachable!("expected only Ingested event but got another event"),
+                        });
+                    entry_stream.insert(stream);
                 },
-                entry = entry_stream.recv() => {
+                entry = entry_stream.next(), if !entry_stream.is_empty() => {
                     match entry {
-                        Ok(entry) => self.send_entry(entry).await?,
-                        Err(broadcast::error::RecvError::Closed) => break,
-                        Err(broadcast::error::RecvError::Lagged(_count)) => {
-                            // TODO: Queue another reconciliation
-                        }
+                        Some(entry) => self.send_entry(entry).await?,
+                        None => break,
                     }
-
                 }
             }
         }
@@ -149,13 +154,9 @@ impl<S: Storage> DataReceiver<S> {
                 message.dynamic_token,
             )
             .await?;
-        self.store.entries().ingest(
-            &authorised_entry,
-            EntryOrigin::Remote {
-                session: self.session_id,
-                channel: EntryChannel::Data,
-            },
-        )?;
+        self.store
+            .entries()
+            .ingest_entry(&authorised_entry, EntryOrigin::Remote(self.session_id))?;
         let (entry, _token) = authorised_entry.into_parts();
         // TODO: handle offset
         self.current_payload.set(

iroh-willow/src/session/reconciler.rs

@@ -29,8 +29,7 @@ use crate::{
         Error, Role, SessionId,
     },
     store::{
-        entry::{EntryChannel, EntryOrigin},
-        traits::{EntryReader, EntryStorage, SplitAction, SplitOpts, Storage},
+        traits::{EntryOrigin, EntryReader, EntryStorage, SplitAction, SplitOpts, Storage},
         Store,
     },
     util::{
@@ -164,12 +163,9 @@ impl<S: Storage> Reconciler<S> {
                     authorised_entry.entry().payload_length(),
                     message.entry.available,
                 )?;
-                self.shared.store.entries().ingest(
+                self.shared.store.entries().ingest_entry(
                     &authorised_entry,
-                    EntryOrigin::Remote {
-                        session: self.shared.session_id,
-                        channel: EntryChannel::Reconciliation,
-                    },
+                    EntryOrigin::Remote(self.shared.session_id),
                 )?;
             }
             ReconciliationMessage::SendPayload(message) => {

iroh-willow/src/session/run.rs

@@ -386,9 +386,6 @@ pub(crate) async fn run_session<S: Storage>(
         .try_join()
         .await;
 
-    // Unsubscribe from the store.
-    store.entries().unsubscribe(&session_id);
-
     // Track if we closed the session by triggering the cancel token, or if the remote peer closed
     // the session by closing the control channel.
     let we_cancelled = close_session_token.is_cancelled();

iroh-willow/src/store.rs

@@ -6,6 +6,7 @@
 
 use anyhow::{anyhow, Context, Result};
 use rand_core::CryptoRngCore;
+use traits::EntryStorage;
 
 use crate::{
     form::{AuthForm, EntryForm, EntryOrForm, SubspaceForm, TimestampForm},
@@ -22,42 +23,40 @@ use crate::{
 use self::auth::{Auth, AuthError};
 use self::traits::Storage;
 
-pub(crate) use self::entry::{EntryOrigin, WatchableEntryStore};
+pub(crate) use self::traits::EntryOrigin;
 
 pub(crate) mod auth;
-pub(crate) mod entry;
 pub mod memory;
 pub mod traits;
 
 /// Storage for the Willow engine.
+///
+/// Wraps a `Storage` instance and adds the [`Auth`] struct that uses the secret and caps storage to provide
+/// authentication when inserting entries.
 #[derive(Debug, Clone)]
 pub(crate) struct Store<S: Storage> {
-    entries: WatchableEntryStore<S::Entries>,
-    secrets: S::Secrets,
-    payloads: S::Payloads,
+    storage: S,
     auth: Auth<S>,
 }
 
 impl<S: Storage> Store<S> {
     pub fn new(storage: S) -> Self {
         Self {
-            entries: WatchableEntryStore::new(storage.entries().clone()),
-            secrets: storage.secrets().clone(),
-            payloads: storage.payloads().clone(),
             auth: Auth::new(storage.secrets().clone(), storage.caps().clone()),
+            storage,
         }
     }
 
-    pub fn entries(&self) -> &WatchableEntryStore<S::Entries> {
-        &self.entries
+    pub fn entries(&self) -> &S::Entries {
+        self.storage.entries()
     }
 
     pub fn secrets(&self) -> &S::Secrets {
-        &self.secrets
+        self.storage.secrets()
     }
 
     pub fn payloads(&self) -> &S::Payloads {
-        &self.payloads
+        self.storage.payloads()
     }
 
     pub fn auth(&self) -> &Auth<S> {
@@ -97,7 +96,7 @@ impl<S: Storage> Store<S> {
         let authorised_entry = AuthorisedEntry::new_unchecked(entry, token);
         let inserted = self
             .entries()
-            .ingest(&authorised_entry, EntryOrigin::Local)?;
+            .ingest_entry(&authorised_entry, EntryOrigin::Local)?;
         Ok((authorised_entry, inserted))
     }
 
@@ -118,7 +117,7 @@ impl<S: Storage> Store<S> {
     /// the provided [`Store`].
     ///
     /// `user_id` must be set to the user who is authenticating the entry.
-    pub async fn form_to_entry(
+    async fn form_to_entry(
         &self,
         form: EntryForm,
         user_id: UserId, // auth: AuthForm,