Minor logging update on observe

jasta · jasta · commit 56c1270919e7 · 2022-05-03T13:26:35.000-07:00
diff --git a/src/app/observe_handler.rs b/src/app/observe_handler.rs
@@ -205,7 +205,8 @@ impl OnObserversChangeDispatcher {
         while let Some(event) = self.rx.next().await {
             match event {
                 ObserverDispatchEvent::OnFirstObserver(observers) => {
-                    log::debug!("entered OnFirstObserver");
+                    let path = observers.relative_path();
+                    log::debug!("entered OnFirstObserver for: {path}");
                     self.resource.on_active(observers).await;
 
                     // TODO: We should probably be mutating the state here to drop all of the observers and
@@ -215,7 +216,7 @@ impl OnObserversChangeDispatcher {
                     // This does mean however that an app cannot signal that for some reason
                     // there's a problem observing the resource and we must forcefully
                     // cleanup with our peers.
-                    log::debug!("exit OnFirstObserver");
+                    log::debug!("exit OnFirstObserver for: {path}");
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -205,7 +205,8 @@ impl OnObserversChangeDispatcher {`
`205`	`205`	`while let Some(event) = self.rx.next().await {`
`206`	`206`	`match event {`
`207`	`207`	`ObserverDispatchEvent::OnFirstObserver(observers) => {`
`208`		`- log::debug!("entered OnFirstObserver");`
	`208`	`+ let path = observers.relative_path();`
	`209`	`+ log::debug!("entered OnFirstObserver for: {path}");`
`209`	`210`	`self.resource.on_active(observers).await;`
`210`	`211`
`211`	`212`	`// TODO: We should probably be mutating the state here to drop all of the observers and`
`@@ -215,7 +216,7 @@ impl OnObserversChangeDispatcher {`
`215`	`216`	`// This does mean however that an app cannot signal that for some reason`
`216`	`217`	`// there's a problem observing the resource and we must forcefully`
`217`	`218`	`// cleanup with our peers.`
`218`		`- log::debug!("exit OnFirstObserver");`
	`219`	`+ log::debug!("exit OnFirstObserver for: {path}");`
`219`	`220`	`}`
`220`	`221`	`}`
`221`	`222`	`}`