Update basics/todo to v4. (#486)

Co-authored-by: Rob Ede <robjtede@icloud.com>

Cargo.lock

@@ -586,23 +586,6 @@ dependencies = [
  "webpki-roots 0.22.2",
 ]
 
-[[package]]
-name = "actix-todo"
-version = "1.0.0"
-dependencies = [
- "actix-files 0.3.0",
- "actix-session 0.4.1",
- "actix-web 3.3.3",
- "diesel",
- "dotenv",
- "env_logger 0.8.4",
- "futures",
- "log",
- "serde 1.0.136",
- "serde_json",
- "tera",
-]
-
 [[package]]
 name = "actix-utils"
 version = "2.0.0"
@@ -6069,6 +6052,23 @@ version = "0.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "cda74da7e1a664f795bb1f8a87ec406fb89a02522cf6e50620d016add6dbbf5c"
 
+[[package]]
+name = "todo"
+version = "1.0.0"
+dependencies = [
+ "actix-files 0.6.0-beta.16",
+ "actix-session 0.5.0-beta.7",
+ "actix-web 4.0.0-rc.1",
+ "diesel",
+ "dotenv",
+ "env_logger 0.9.0",
+ "futures",
+ "log",
+ "serde 1.0.136",
+ "serde_json",
+ "tera",
+]
+
 [[package]]
 name = "tokio"
 version = "0.2.25"

basics/todo/Cargo.toml

@@ -1,21 +1,18 @@
 [package]
-name = "actix-todo"
+name = "todo"
 version = "1.0.0"
 edition = "2021"
 
 [dependencies]
-actix-web = "3"
-actix-files = "0.3"
-actix-session = "0.4"
+actix-web = "4.0.0-rc.1"
+actix-files = "0.6.0-beta.15"
+actix-session = "0.5.0-beta.7"
 
+diesel = { version = "1.3.2", features = ["postgres", "r2d2"] }
 dotenv = "0.15"
-env_logger = "0.8"
-futures = "0.3.1"
-log = "0.4.3"
-serde = { version = "1.0.69", features = ["derive"] }
-serde_json = "1.0.22"
-tera = "1.5.0"
-
-[dependencies.diesel]
-features = ["postgres", "r2d2"]
-version = "1.3.2"
+env_logger = "0.9"
+futures = "0.3.7"
+log = "0.4"
+serde = { version = "1", features = ["derive"] }
+serde_json = "1"
+tera = "1.5"

basics/todo/README.md

@@ -45,4 +45,4 @@ To run the application execute:
 cargo run
 ```
 
-Then to view it in your browser navigate to: [http://localhost:8088/](http://localhost:8088/)
+Then to view it in your browser navigate to: [http://localhost:8080/](http://localhost:8080/)

basics/todo/src/api.rs

@@ -1,12 +1,15 @@
 use actix_files::NamedFile;
 use actix_session::Session;
-use actix_web::middleware::errhandlers::ErrorHandlerResponse;
-use actix_web::{dev, error, http, web, Error, HttpResponse, Result};
+use actix_web::{
+    dev, error, http, middleware::ErrorHandlerResponse, web, Error, HttpResponse, Result,
+};
 use serde::Deserialize;
 use tera::{Context, Tera};
 
-use crate::db;
-use crate::session::{self, FlashMessage};
+use crate::{
+    db,
+    session::{self, FlashMessage},
+};
 
 pub async fn index(
     pool: web::Data<db::PgPool>,
@@ -50,7 +53,8 @@ pub async fn create(
         Ok(redirect_to("/"))
     } else {
         web::block(move || db::create_task(params.into_inner().description, &pool))
-            .await?;
+            .await?
+            .map_err(error::ErrorInternalServerError)?;
         session::set_flash(&session, FlashMessage::success("Task successfully added"))?;
         Ok(redirect_to("/"))
     }
@@ -86,7 +90,9 @@ async fn toggle(
     pool: web::Data<db::PgPool>,
     params: web::Path<UpdateParams>,
 ) -> Result<HttpResponse, Error> {
-    web::block(move || db::toggle_task(params.id, &pool)).await?;
+    web::block(move || db::toggle_task(params.id, &pool))
+        .await?
+        .map_err(error::ErrorInternalServerError)?;
     Ok(redirect_to("/"))
 }
 
@@ -95,33 +101,33 @@ async fn delete(
     params: web::Path<UpdateParams>,
     session: Session,
 ) -> Result<HttpResponse, Error> {
-    web::block(move || db::delete_task(params.id, &pool)).await?;
+    web::block(move || db::delete_task(params.id, &pool))
+        .await?
+        .map_err(error::ErrorInternalServerError)?;
     session::set_flash(&session, FlashMessage::success("Task was deleted."))?;
     Ok(redirect_to("/"))
 }
 
 fn redirect_to(location: &str) -> HttpResponse {
     HttpResponse::Found()
-        .header(http::header::LOCATION, location)
+        .append_header((http::header::LOCATION, location))
         .finish()
 }
 
 pub fn bad_request<B>(res: dev::ServiceResponse<B>) -> Result<ErrorHandlerResponse<B>> {
     let new_resp = NamedFile::open("static/errors/400.html")?
         .set_status_code(res.status())
-        .into_response(res.request())?;
-    Ok(ErrorHandlerResponse::Response(
-        res.into_response(new_resp.into_body()),
-    ))
+        .into_response(res.request())
+        .map_into_right_body();
+    Ok(ErrorHandlerResponse::Response(res.into_response(new_resp)))
 }
 
 pub fn not_found<B>(res: dev::ServiceResponse<B>) -> Result<ErrorHandlerResponse<B>> {
     let new_resp = NamedFile::open("static/errors/404.html")?
         .set_status_code(res.status())
-        .into_response(res.request())?;
-    Ok(ErrorHandlerResponse::Response(
-        res.into_response(new_resp.into_body()),
-    ))
+        .into_response(res.request())
+        .map_into_right_body();
+    Ok(ErrorHandlerResponse::Response(res.into_response(new_resp)))
 }
 
 pub fn internal_server_error<B>(
@@ -129,8 +135,7 @@ pub fn internal_server_error<B>(
 ) -> Result<ErrorHandlerResponse<B>> {
     let new_resp = NamedFile::open("static/errors/500.html")?
         .set_status_code(res.status())
-        .into_response(res.request())?;
-    Ok(ErrorHandlerResponse::Response(
-        res.into_response(new_resp.into_body()),
-    ))
+        .into_response(res.request())
+        .map_into_right_body();
+    Ok(ErrorHandlerResponse::Response(res.into_response(new_resp)))
 }

basics/todo/src/db.rs

@@ -1,7 +1,9 @@
-use std::ops::Deref;
+use std::ops::Deref as _;
 
-use diesel::pg::PgConnection;
-use diesel::r2d2::{ConnectionManager, Pool, PoolError, PooledConnection};
+use diesel::{
+    pg::PgConnection,
+    r2d2::{ConnectionManager, Pool, PoolError, PooledConnection},
+};
 
 use crate::model::{NewTask, Task};
 

basics/todo/src/main.rs

@@ -1,14 +1,15 @@
 #[macro_use]
 extern crate diesel;
-#[macro_use]
-extern crate log;
 
 use std::{env, io};
 
-use actix_files as fs;
+use actix_files::Files;
 use actix_session::CookieSession;
-use actix_web::middleware::{errhandlers::ErrorHandlers, Logger};
-use actix_web::{http, web, App, HttpServer};
+use actix_web::{
+    http,
+    middleware::{ErrorHandlers, Logger},
+    web, App, HttpServer,
+};
 use dotenv::dotenv;
 use tera::Tera;
 
@@ -24,22 +25,18 @@ static SESSION_SIGNING_KEY: &[u8] = &[0; 32];
 async fn main() -> io::Result<()> {
     dotenv().ok();
 
-    env::set_var("RUST_LOG", "actix_todo=debug,actix_web=info");
-    env_logger::init();
+    env_logger::init_from_env(env_logger::Env::new().default_filter_or("info"));
 
     let database_url = env::var("DATABASE_URL").expect("DATABASE_URL must be set");
     let pool = db::init_pool(&database_url).expect("Failed to create pool");
 
-    let app = move || {
-        debug!("Constructing the App");
+    log::info!("starting HTTP serer at http://localhost:8080");
 
-        let mut templates = match Tera::new("templates/**/*") {
-            Ok(t) => t,
-            Err(e) => {
-                println!("Parsing error(s): {}", e);
-                ::std::process::exit(1);
-            }
-        };
+    HttpServer::new(move || {
+        log::debug!("Constructing the App");
+
+        let mut templates =
+            Tera::new("templates/**/*").expect("errors in tera templates");
         templates.autoescape_on(vec!["tera"]);
 
         let session_store = CookieSession::signed(SESSION_SIGNING_KEY).secure(false);
@@ -53,17 +50,17 @@ async fn main() -> io::Result<()> {
             .handler(http::StatusCode::NOT_FOUND, api::not_found);
 
         App::new()
-            .data(templates)
-            .data(pool.clone())
+            .app_data(templates)
+            .app_data(pool.clone())
             .wrap(Logger::default())
             .wrap(session_store)
             .wrap(error_handlers)
             .service(web::resource("/").route(web::get().to(api::index)))
             .service(web::resource("/todo").route(web::post().to(api::create)))
             .service(web::resource("/todo/{id}").route(web::post().to(api::update)))
-            .service(fs::Files::new("/static", "static/"))
-    };
-
-    debug!("Starting server");
-    HttpServer::new(app).bind("localhost:8088")?.run().await
+            .service(Files::new("/static", "./static/"))
+    })
+    .bind(("127.0.0.1", 8088))?
+    .run()
+    .await
 }

basics/todo/src/session.rs

@@ -5,7 +5,7 @@ use serde::{Deserialize, Serialize};
 const FLASH_KEY: &str = "flash";
 
 pub fn set_flash(session: &Session, flash: FlashMessage) -> Result<()> {
-    session.set(FLASH_KEY, flash)
+    session.insert(FLASH_KEY, flash)
 }
 
 pub fn get_flash(session: &Session) -> Result<Option<FlashMessage>> {