wrap AppData in Data.

This seems like a weird gotcha in ActixWeb v4. Why not only accept a Data instead of letting this fail at runtime!?
NfNitLoop · Sep 26, 2022 · 433d28b · 433d28b
1 parent f02e98d
commit 433d28b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "feoblog"
-version = "0.7.0-rc3"
+version = "0.7.0-rc5"
 authors = ["Cody Casterline <cody.casterline@gmail.com>"]
 edition = "2018"
 

diff --git a/src/server.rs b/src/server.rs
@@ -57,11 +57,14 @@ pub(crate) fn serve(command: ServeCommand) -> Result<(), anyhow::Error> {
     };
 
     let app_factory = move || {
+        let data = Data::new(
+            AppData{
+                backend_factory: factory_box.factory.dyn_clone(),
+            }
+        );
         let mut app = App::new()
             .wrap(actix_web::middleware::Logger::default())
-            .app_data(AppData{
-                backend_factory: factory_box.factory.dyn_clone(),
-            })
+            .app_data(data)
             .configure(routes)
         ;