From bfbbf96c41fecab95424226db2a551cd03b703cf Mon Sep 17 00:00:00 2001 From: Luca Palmieri Date: Sat, 5 Dec 2020 17:07:31 +0000 Subject: [PATCH] Chapter 05 (snapshot) --- chapter05/.dockerignore => .dockerignore | 0 chapter03-1/.env => .env | 0 .github/workflows/general.yml | 9 +- Cargo.lock | 93 +++--------- Cargo.toml | 36 ++++- chapter05/Dockerfile => Dockerfile | 4 +- chapter03-0/Cargo.toml | 21 --- chapter03-0/src/lib.rs | 14 -- chapter03-0/src/main.rs | 8 - chapter03-0/tests/health_check.rs | 31 ---- chapter03-1/Cargo.toml | 26 ---- chapter03-1/configuration.yaml | 7 - chapter03-1/src/configuration.rs | 38 ----- chapter03-1/src/lib.rs | 4 - chapter03-1/src/main.rs | 23 --- chapter03-1/src/routes/subscriptions.rs | 33 ---- chapter03-1/src/startup.rs | 19 --- chapter03-1/tests/health_check.rs | 129 ---------------- chapter04/.env | 1 - chapter04/Cargo.toml | 33 ---- chapter04/configuration.yaml | 7 - ...00823135036_create_subscriptions_table.sql | 8 - chapter04/scripts/init_db.sh | 40 ----- chapter04/src/configuration.rs | 38 ----- chapter04/src/main.rs | 27 ---- chapter04/src/routes/health_check.rs | 5 - chapter04/src/routes/mod.rs | 5 - chapter04/tests/health_check.rs | 143 ------------------ chapter05/.env | 1 - chapter05/Cargo.toml | 34 ----- ...00823135036_create_subscriptions_table.sql | 8 - chapter05/scripts/init_db.sh | 40 ----- chapter05/src/lib.rs | 5 - chapter05/src/routes/health_check.rs | 5 - chapter05/src/routes/mod.rs | 5 - chapter05/src/routes/subscriptions.rs | 52 ------- chapter05/src/startup.rs | 21 --- chapter05/src/telemetry.rs | 29 ---- .../configuration => configuration}/base.yaml | 0 .../local.yaml | 0 .../production.yaml | 0 ...00823135036_create_subscriptions_table.sql | 0 {chapter03-1/scripts => scripts}/init_db.sh | 0 chapter05/spec.yaml => spec.yaml | 4 +- chapter05/sqlx-data.json => sqlx-data.json | 0 {chapter05/src => src}/configuration.rs | 0 {chapter04/src => src}/lib.rs | 0 {chapter05/src => src}/main.rs | 6 +- .../src => src}/routes/health_check.rs | 0 {chapter03-1/src => src}/routes/mod.rs | 0 .../src => src}/routes/subscriptions.rs | 0 {chapter04/src => src}/startup.rs | 0 {chapter04/src => src}/telemetry.rs | 0 {chapter05/tests => tests}/health_check.rs | 6 +- 54 files changed, 75 insertions(+), 943 deletions(-) rename chapter05/.dockerignore => .dockerignore (100%) rename chapter03-1/.env => .env (100%) rename chapter05/Dockerfile => Dockerfile (91%) delete mode 100644 chapter03-0/Cargo.toml delete mode 100644 chapter03-0/src/lib.rs delete mode 100644 chapter03-0/src/main.rs delete mode 100644 chapter03-0/tests/health_check.rs delete mode 100644 chapter03-1/Cargo.toml delete mode 100644 chapter03-1/configuration.yaml delete mode 100644 chapter03-1/src/configuration.rs delete mode 100644 chapter03-1/src/lib.rs delete mode 100644 chapter03-1/src/main.rs delete mode 100644 chapter03-1/src/routes/subscriptions.rs delete mode 100644 chapter03-1/src/startup.rs delete mode 100644 chapter03-1/tests/health_check.rs delete mode 100644 chapter04/.env delete mode 100644 chapter04/Cargo.toml delete mode 100644 chapter04/configuration.yaml delete mode 100644 chapter04/migrations/20200823135036_create_subscriptions_table.sql delete mode 100755 chapter04/scripts/init_db.sh delete mode 100644 chapter04/src/configuration.rs delete mode 100644 chapter04/src/main.rs delete mode 100644 chapter04/src/routes/health_check.rs delete mode 100644 chapter04/src/routes/mod.rs delete mode 100644 chapter04/tests/health_check.rs delete mode 100644 chapter05/.env delete mode 100644 chapter05/Cargo.toml delete mode 100644 chapter05/migrations/20200823135036_create_subscriptions_table.sql delete mode 100755 chapter05/scripts/init_db.sh delete mode 100644 chapter05/src/lib.rs delete mode 100644 chapter05/src/routes/health_check.rs delete mode 100644 chapter05/src/routes/mod.rs delete mode 100644 chapter05/src/routes/subscriptions.rs delete mode 100644 chapter05/src/startup.rs delete mode 100644 chapter05/src/telemetry.rs rename {chapter05/configuration => configuration}/base.yaml (100%) rename {chapter05/configuration => configuration}/local.yaml (100%) rename {chapter05/configuration => configuration}/production.yaml (100%) rename {chapter03-1/migrations => migrations}/20200823135036_create_subscriptions_table.sql (100%) rename {chapter03-1/scripts => scripts}/init_db.sh (100%) rename chapter05/spec.yaml => spec.yaml (95%) rename chapter05/sqlx-data.json => sqlx-data.json (100%) rename {chapter05/src => src}/configuration.rs (100%) rename {chapter04/src => src}/lib.rs (100%) rename {chapter05/src => src}/main.rs (84%) rename {chapter03-1/src => src}/routes/health_check.rs (100%) rename {chapter03-1/src => src}/routes/mod.rs (100%) rename {chapter04/src => src}/routes/subscriptions.rs (100%) rename {chapter04/src => src}/startup.rs (100%) rename {chapter04/src => src}/telemetry.rs (100%) rename {chapter05/tests => tests}/health_check.rs (96%) diff --git a/chapter05/.dockerignore b/.dockerignore similarity index 100% rename from chapter05/.dockerignore rename to .dockerignore diff --git a/chapter03-1/.env b/.env similarity index 100% rename from chapter03-1/.env rename to .env diff --git a/.github/workflows/general.yml b/.github/workflows/general.yml index e417859..f7207a7 100644 --- a/.github/workflows/general.yml +++ b/.github/workflows/general.yml @@ -1,6 +1,13 @@ name: Rust -on: [push, pull_request] +on: + # NB: this differs from the book's project! + # These settings allow us to run this specific CI pipeline for PRs against + # this specific branch (a.k.a. book chapter). + pull_request: + types: [ opened, synchronize, reopened ] + branches: + - root-chapter-05 env: CARGO_TERM_COLOR: always diff --git a/Cargo.lock b/Cargo.lock index 9bf71a9..7b76292 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -473,76 +473,6 @@ version = "1.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "baf1de4339761588bc0619e3cbc0120ee582ebb74b53b4efbf79117bd2da40fd" -[[package]] -name = "chapter03-0" -version = "0.1.0" -dependencies = [ - "actix-rt", - "actix-web", - "reqwest", - "tokio", -] - -[[package]] -name = "chapter03-1" -version = "0.1.0" -dependencies = [ - "actix-rt", - "actix-web", - "chrono", - "config", - "reqwest", - "serde", - "sqlx", - "tokio", - "uuid", -] - -[[package]] -name = "chapter04" -version = "0.1.0" -dependencies = [ - "actix-rt", - "actix-web", - "chrono", - "config", - "lazy_static", - "reqwest", - "serde", - "sqlx", - "tokio", - "tracing", - "tracing-actix-web", - "tracing-bunyan-formatter", - "tracing-futures", - "tracing-log", - "tracing-subscriber", - "uuid", -] - -[[package]] -name = "chapter05" -version = "0.1.0" -dependencies = [ - "actix-rt", - "actix-web", - "chrono", - "config", - "lazy_static", - "reqwest", - "serde", - "serde-aux", - "sqlx", - "tokio", - "tracing", - "tracing-actix-web", - "tracing-bunyan-formatter", - "tracing-futures", - "tracing-log", - "tracing-subscriber", - "uuid", -] - [[package]] name = "chrono" version = "0.4.19" @@ -2730,3 +2660,26 @@ checksum = "39f0c922f1a334134dc2f7a8b67dc5d25f0735263feec974345ff706bcf20b0d" dependencies = [ "linked-hash-map", ] + +[[package]] +name = "zero2prod" +version = "0.1.0" +dependencies = [ + "actix-rt", + "actix-web", + "chrono", + "config", + "lazy_static", + "reqwest", + "serde", + "serde-aux", + "sqlx", + "tokio", + "tracing", + "tracing-actix-web", + "tracing-bunyan-formatter", + "tracing-futures", + "tracing-log", + "tracing-subscriber", + "uuid", +] diff --git a/Cargo.toml b/Cargo.toml index 7b05a59..278d87c 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -1,2 +1,34 @@ -[workspace] -members = ["chapter03-0", "chapter03-1", "chapter04", "chapter05"] +[package] +name = "zero2prod" +version = "0.1.0" +authors = ["LukeMathWalker "] +edition = "2018" + +# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html +[lib] +path = "src/lib.rs" + +[[bin]] +path = "src/main.rs" +name = "zero2prod" + +[dependencies] +actix-web = "3.0.0" +actix-rt = "1.1.1" +tokio = "0.2.22" +serde = "1.0.115" +config = { version = "0.10.1", default-features = false, features = ["yaml"] } +sqlx = { version = "0.4.0-beta.1", default-features = false, features = [ "runtime-tokio", "macros", "postgres", "uuid", "chrono", "migrate", "offline"] } +uuid = { version = "0.8.1", features = ["v4"] } +chrono = "0.4.15" +tracing = "0.1.19" +tracing-futures = "0.2.4" +tracing-subscriber = { version = "0.2.12", features = ["registry", "env-filter"] } +tracing-bunyan-formatter = "0.1.6" +tracing-log = "0.1.1" +tracing-actix-web = "0.2.0" +serde-aux = "1.0.1" + +[dev-dependencies] +reqwest = { version = "0.10.7", features = ["json"] } +lazy_static = "1.4.0" diff --git a/chapter05/Dockerfile b/Dockerfile similarity index 91% rename from chapter05/Dockerfile rename to Dockerfile index 9b72852..e90c7c6 100644 --- a/chapter05/Dockerfile +++ b/Dockerfile @@ -24,7 +24,7 @@ COPY --from=cacher /usr/local/cargo /usr/local/cargo COPY . . # Build our application, leveraging the cached deps! ENV SQLX_OFFLINE true -RUN cargo build --release --bin chapter05 +RUN cargo build --release --bin zero2prod FROM debian:buster-slim AS runtime WORKDIR app @@ -32,7 +32,7 @@ RUN apt-get update -y \ && apt-get install -y --no-install-recommends openssl \ # Clean up && apt-get autoremove -y && apt-get clean -y && rm -rf /var/lib/apt/lists/* -COPY --from=builder /app/target/release/chapter05 zero2prod +COPY --from=builder /app/target/release/zero2prod zero2prod COPY configuration configuration ENV APP_ENVIRONMENT production ENTRYPOINT ["./zero2prod"] diff --git a/chapter03-0/Cargo.toml b/chapter03-0/Cargo.toml deleted file mode 100644 index 4e5dab3..0000000 --- a/chapter03-0/Cargo.toml +++ /dev/null @@ -1,21 +0,0 @@ -[package] -name = "chapter03-0" -version = "0.1.0" -authors = ["LukeMathWalker "] -edition = "2018" - -# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html -[lib] -path = "src/lib.rs" - -[[bin]] -path = "src/main.rs" -name = "chapter03-0" - -[dependencies] -actix-web = "3.0.0" -actix-rt = "1.1.1" -tokio = "0.2.22" - -[dev-dependencies] -reqwest = "0.10.7" diff --git a/chapter03-0/src/lib.rs b/chapter03-0/src/lib.rs deleted file mode 100644 index d4321d5..0000000 --- a/chapter03-0/src/lib.rs +++ /dev/null @@ -1,14 +0,0 @@ -use actix_web::dev::Server; -use actix_web::{web, App, HttpResponse, HttpServer}; -use std::net::TcpListener; - -async fn health_check() -> HttpResponse { - HttpResponse::Ok().finish() -} - -pub fn run(listener: TcpListener) -> Result { - let server = HttpServer::new(|| App::new().route("/health_check", web::get().to(health_check))) - .listen(listener)? - .run(); - Ok(server) -} diff --git a/chapter03-0/src/main.rs b/chapter03-0/src/main.rs deleted file mode 100644 index cb6021e..0000000 --- a/chapter03-0/src/main.rs +++ /dev/null @@ -1,8 +0,0 @@ -use chapter03_0::run; -use std::net::TcpListener; - -#[actix_rt::main] -async fn main() -> std::io::Result<()> { - let address = TcpListener::bind("127.0.0.1:8000")?; - run(address)?.await -} diff --git a/chapter03-0/tests/health_check.rs b/chapter03-0/tests/health_check.rs deleted file mode 100644 index 98596a1..0000000 --- a/chapter03-0/tests/health_check.rs +++ /dev/null @@ -1,31 +0,0 @@ -use chapter03_0::run; -use std::net::TcpListener; - -fn spawn_app() -> String { - let listener = TcpListener::bind("127.0.0.1:0").expect("Failed to bind random port"); - // We retrieve the port assigned to us by the OS - let port = listener.local_addr().unwrap().port(); - let server = run(listener).expect("Failed to bind address"); - let _ = tokio::spawn(server); - // We return the application address to the caller! - format!("http://127.0.0.1:{}", port) -} - -#[actix_rt::test] -async fn health_check_works() { - // Arrange - let address = spawn_app(); - let client = reqwest::Client::new(); - - // Act - let response = client - // Use the returned application address - .get(&format!("{}/health_check", &address)) - .send() - .await - .expect("Failed to execute request."); - - // Assert - assert!(response.status().is_success()); - assert_eq!(Some(0), response.content_length()); -} diff --git a/chapter03-1/Cargo.toml b/chapter03-1/Cargo.toml deleted file mode 100644 index 2164985..0000000 --- a/chapter03-1/Cargo.toml +++ /dev/null @@ -1,26 +0,0 @@ -[package] -name = "chapter03-1" -version = "0.1.0" -authors = ["LukeMathWalker "] -edition = "2018" - -# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html -[lib] -path = "src/lib.rs" - -[[bin]] -path = "src/main.rs" -name = "chapter03-1" - -[dependencies] -actix-web = "3.0.0" -actix-rt = "1.1.1" -tokio = "0.2.22" -serde = "1.0.115" -config = { version = "0.10.1", default-features = false, features = ["yaml"] } -sqlx = { version = "0.4.0-beta.1", default-features = false, features = [ "runtime-tokio", "macros", "postgres", "uuid", "chrono", "migrate"] } -uuid = { version = "0.8.1", features = ["v4"] } -chrono = "0.4.15" - -[dev-dependencies] -reqwest = { version = "0.10.7", features = ["json"] } diff --git a/chapter03-1/configuration.yaml b/chapter03-1/configuration.yaml deleted file mode 100644 index 04dc5ef..0000000 --- a/chapter03-1/configuration.yaml +++ /dev/null @@ -1,7 +0,0 @@ -application_port: 8000 -database: - host: "localhost" - port: 5432 - username: "postgres" - password: "password" - database_name: "newsletter" \ No newline at end of file diff --git a/chapter03-1/src/configuration.rs b/chapter03-1/src/configuration.rs deleted file mode 100644 index 3ad6da7..0000000 --- a/chapter03-1/src/configuration.rs +++ /dev/null @@ -1,38 +0,0 @@ -#[derive(serde::Deserialize)] -pub struct Settings { - pub database: DatabaseSettings, - pub application_port: u16, -} - -#[derive(serde::Deserialize)] -pub struct DatabaseSettings { - pub username: String, - pub password: String, - pub port: u16, - pub host: String, - pub database_name: String, -} - -impl DatabaseSettings { - pub fn connection_string(&self) -> String { - format!( - "postgres://{}:{}@{}:{}/{}", - self.username, self.password, self.host, self.port, self.database_name - ) - } - - pub fn connection_string_without_db(&self) -> String { - format!( - "postgres://{}:{}@{}:{}", - self.username, self.password, self.host, self.port - ) - } -} - -pub fn get_configuration() -> Result { - let mut settings = config::Config::default(); - - settings.merge(config::File::with_name("configuration"))?; - - settings.try_into() -} diff --git a/chapter03-1/src/lib.rs b/chapter03-1/src/lib.rs deleted file mode 100644 index b477f22..0000000 --- a/chapter03-1/src/lib.rs +++ /dev/null @@ -1,4 +0,0 @@ -#![allow(clippy::toplevel_ref_arg)] -pub mod configuration; -pub mod routes; -pub mod startup; diff --git a/chapter03-1/src/main.rs b/chapter03-1/src/main.rs deleted file mode 100644 index b4f54f4..0000000 --- a/chapter03-1/src/main.rs +++ /dev/null @@ -1,23 +0,0 @@ -use chapter03_1::configuration::get_configuration; -use chapter03_1::startup::run; -use sqlx::postgres::PgPool; -use std::net::TcpListener; - -#[actix_rt::main] -async fn main() -> std::io::Result<()> { - let configuration = get_configuration().expect("Failed to read configuration."); - let connection_pool = PgPool::connect(&configuration.database.connection_string()) - .await - .expect("Failed to connect to Postgres."); - - // Here we choose to bind explicitly to localhost, 127.0.0.1, for security - // reasons. This binding may cause issues in some environments. For example, - // it causes connectivity issues running in WSL2, where you cannot reach the - // server when it is bound to WSL2's localhost interface. As a workaround, - // you can choose to bind to all interfaces, 0.0.0.0, instead, but be aware - // of the security implications when you expose the server on all interfaces. - let address = format!("127.0.0.1:{}", configuration.application_port); - let listener = TcpListener::bind(address)?; - run(listener, connection_pool)?.await?; - Ok(()) -} diff --git a/chapter03-1/src/routes/subscriptions.rs b/chapter03-1/src/routes/subscriptions.rs deleted file mode 100644 index 8db4b58..0000000 --- a/chapter03-1/src/routes/subscriptions.rs +++ /dev/null @@ -1,33 +0,0 @@ -use actix_web::{web, HttpResponse}; -use chrono::Utc; -use sqlx::PgPool; -use uuid::Uuid; - -#[derive(serde::Deserialize)] -pub struct FormData { - email: String, - name: String, -} - -pub async fn subscribe( - form: web::Form, - pool: web::Data, -) -> Result { - sqlx::query!( - r#" - INSERT INTO subscriptions (id, email, name, subscribed_at) - VALUES ($1, $2, $3, $4) - "#, - Uuid::new_v4(), - form.email, - form.name, - Utc::now() - ) - .execute(pool.as_ref()) - .await - .map_err(|e| { - println!("Failed to execute query: {}", e); - HttpResponse::InternalServerError().finish() - })?; - Ok(HttpResponse::Ok().finish()) -} diff --git a/chapter03-1/src/startup.rs b/chapter03-1/src/startup.rs deleted file mode 100644 index 9f35c15..0000000 --- a/chapter03-1/src/startup.rs +++ /dev/null @@ -1,19 +0,0 @@ -use crate::routes::{health_check, subscribe}; -use actix_web::dev::Server; -use actix_web::web::Data; -use actix_web::{web, App, HttpServer}; -use sqlx::PgPool; -use std::net::TcpListener; - -pub fn run(listener: TcpListener, db_pool: PgPool) -> Result { - let db_pool = Data::new(db_pool); - let server = HttpServer::new(move || { - App::new() - .route("/health_check", web::get().to(health_check)) - .route("/subscriptions", web::post().to(subscribe)) - .app_data(db_pool.clone()) - }) - .listen(listener)? - .run(); - Ok(server) -} diff --git a/chapter03-1/tests/health_check.rs b/chapter03-1/tests/health_check.rs deleted file mode 100644 index 61de9c6..0000000 --- a/chapter03-1/tests/health_check.rs +++ /dev/null @@ -1,129 +0,0 @@ -use chapter03_1::configuration::{get_configuration, DatabaseSettings}; -use chapter03_1::startup::run; -use sqlx::{Connection, Executor, PgConnection, PgPool}; -use std::net::TcpListener; -use uuid::Uuid; - -pub struct TestApp { - pub address: String, - pub db_pool: PgPool, -} - -async fn spawn_app() -> TestApp { - let listener = TcpListener::bind("127.0.0.1:0").expect("Failed to bind random port"); - // We retrieve the port assigned to us by the OS - let port = listener.local_addr().unwrap().port(); - let address = format!("http://127.0.0.1:{}", port); - - let mut configuration = get_configuration().expect("Failed to read configuration."); - configuration.database.database_name = Uuid::new_v4().to_string(); - let connection_pool = configure_database(&configuration.database).await; - - let server = run(listener, connection_pool.clone()).expect("Failed to bind address"); - let _ = tokio::spawn(server); - TestApp { - address, - db_pool: connection_pool, - } -} - -pub async fn configure_database(config: &DatabaseSettings) -> PgPool { - // Create database - let mut connection = PgConnection::connect(&config.connection_string_without_db()) - .await - .expect("Failed to connect to Postgres"); - connection - .execute(&*format!(r#"CREATE DATABASE "{}";"#, config.database_name)) - .await - .expect("Failed to create database."); - - // Migrate database - let connection_pool = PgPool::connect(&config.connection_string()) - .await - .expect("Failed to connect to Postgres."); - sqlx::migrate!("./migrations") - .run(&connection_pool) - .await - .expect("Failed to migrate the database"); - - connection_pool -} - -#[actix_rt::test] -async fn health_check_works() { - // Arrange - let app = spawn_app().await; - let client = reqwest::Client::new(); - - // Act - let response = client - // Use the returned application address - .get(&format!("{}/health_check", &app.address)) - .send() - .await - .expect("Failed to execute request."); - - // Assert - assert!(response.status().is_success()); - assert_eq!(Some(0), response.content_length()); -} - -#[actix_rt::test] -async fn subscribe_returns_a_200_for_valid_form_data() { - // Arrange - let app = spawn_app().await; - let client = reqwest::Client::new(); - let body = "name=le%20guin&email=ursula_le_guin%40gmail.com"; - - // Act - let response = client - .post(&format!("{}/subscriptions", &app.address)) - .header("Content-Type", "application/x-www-form-urlencoded") - .body(body) - .send() - .await - .expect("Failed to execute request."); - - // Assert - assert_eq!(200, response.status().as_u16()); - - let saved = sqlx::query!("SELECT email, name FROM subscriptions",) - .fetch_one(&app.db_pool) - .await - .expect("Failed to fetch saved subscription."); - - assert_eq!(saved.email, "ursula_le_guin@gmail.com"); - assert_eq!(saved.name, "le guin"); -} - -#[actix_rt::test] -async fn subscribe_returns_a_400_when_data_is_missing() { - // Arrange - let app = spawn_app().await; - let client = reqwest::Client::new(); - let test_cases = vec![ - ("name=le%20guin", "missing the email"), - ("email=ursula_le_guin%40gmail.com", "missing the name"), - ("", "missing both name and email"), - ]; - - for (invalid_body, error_message) in test_cases { - // Act - let response = client - .post(&format!("{}/subscriptions", &app.address)) - .header("Content-Type", "application/x-www-form-urlencoded") - .body(invalid_body) - .send() - .await - .expect("Failed to execute request."); - - // Assert - assert_eq!( - 400, - response.status().as_u16(), - // Additional customised error message on test failure - "The API did not fail with 400 Bad Request when the payload was {}.", - error_message - ); - } -} diff --git a/chapter04/.env b/chapter04/.env deleted file mode 100644 index 88cfb53..0000000 --- a/chapter04/.env +++ /dev/null @@ -1 +0,0 @@ -DATABASE_URL="postgres://postgres:password@localhost:5432/newsletter" diff --git a/chapter04/Cargo.toml b/chapter04/Cargo.toml deleted file mode 100644 index 01edb36..0000000 --- a/chapter04/Cargo.toml +++ /dev/null @@ -1,33 +0,0 @@ -[package] -name = "chapter04" -version = "0.1.0" -authors = ["LukeMathWalker "] -edition = "2018" - -# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html -[lib] -path = "src/lib.rs" - -[[bin]] -path = "src/main.rs" -name = "chapter04" - -[dependencies] -actix-web = "3.0.0" -actix-rt = "1.1.1" -tokio = "0.2.22" -serde = "1.0.115" -config = { version = "0.10.1", default-features = false, features = ["yaml"] } -sqlx = { version = "0.4.0-beta.1", default-features = false, features = [ "runtime-tokio", "macros", "postgres", "uuid", "chrono", "migrate"] } -uuid = { version = "0.8.1", features = ["v4"] } -chrono = "0.4.15" -tracing = "0.1.19" -tracing-futures = "0.2.4" -tracing-subscriber = { version = "0.2.12", features = ["registry", "env-filter"] } -tracing-bunyan-formatter = "0.1.6" -tracing-log = "0.1.1" -tracing-actix-web = "0.2.0" - -[dev-dependencies] -reqwest = { version = "0.10.7", features = ["json"] } -lazy_static = "1.4.0" diff --git a/chapter04/configuration.yaml b/chapter04/configuration.yaml deleted file mode 100644 index 04dc5ef..0000000 --- a/chapter04/configuration.yaml +++ /dev/null @@ -1,7 +0,0 @@ -application_port: 8000 -database: - host: "localhost" - port: 5432 - username: "postgres" - password: "password" - database_name: "newsletter" \ No newline at end of file diff --git a/chapter04/migrations/20200823135036_create_subscriptions_table.sql b/chapter04/migrations/20200823135036_create_subscriptions_table.sql deleted file mode 100644 index 2c0d262..0000000 --- a/chapter04/migrations/20200823135036_create_subscriptions_table.sql +++ /dev/null @@ -1,8 +0,0 @@ --- Create Subscriptions Table -CREATE TABLE subscriptions( - id uuid NOT NULL, - PRIMARY KEY (id), - email TEXT NOT NULL UNIQUE, - name TEXT NOT NULL, - subscribed_at timestamptz NOT NULL -); \ No newline at end of file diff --git a/chapter04/scripts/init_db.sh b/chapter04/scripts/init_db.sh deleted file mode 100755 index cc1df04..0000000 --- a/chapter04/scripts/init_db.sh +++ /dev/null @@ -1,40 +0,0 @@ -#!/usr/bin/env bash -set -x -set -eo pipefail - -# Check if a custom user has been set, otherwise default to 'postgres' -DB_USER=${POSTGRES_USER:=postgres} -# Check if a custom password has been set, otherwise default to 'password' -DB_PASSWORD="${POSTGRES_PASSWORD:=password}" -# Check if a custom password has been set, otherwise default to 'newsletter' -DB_NAME="${POSTGRES_DB:=newsletter}" -# Check if a custom port has been set, otherwise default to '5432' -DB_PORT="${POSTGRES_PORT:=5432}" - -# Allow to skip Docker if a dockerized Postgres database is already running -if [[ -z "${SKIP_DOCKER}" ]] -then - # Launch postgres using Docker - docker run \ - -e POSTGRES_USER=${DB_USER} \ - -e POSTGRES_PASSWORD=${DB_PASSWORD} \ - -e POSTGRES_DB=${DB_NAME} \ - -p "${DB_PORT}":5432 \ - -d postgres \ - postgres -N 1000 - # ^ Increased maximum number of connections for testing purposes -fi - -# Keep pinging Postgres until it's ready to accept commands -until PGPASSWORD="${DB_PASSWORD}" psql -h "localhost" -U "${DB_USER}" -p "${DB_PORT}" -d "postgres" -c '\q'; do - >&2 echo "Postgres is still unavailable - sleeping" - sleep 1 -done - ->&2 echo "Postgres is up and running on port ${DB_PORT} - running migrations now!" - -export DATABASE_URL=postgres://${DB_USER}:${DB_PASSWORD}@localhost:${DB_PORT}/${DB_NAME} -sqlx database create -sqlx migrate run - ->&2 echo "Postgres has been migrated, ready to go!" diff --git a/chapter04/src/configuration.rs b/chapter04/src/configuration.rs deleted file mode 100644 index 3ad6da7..0000000 --- a/chapter04/src/configuration.rs +++ /dev/null @@ -1,38 +0,0 @@ -#[derive(serde::Deserialize)] -pub struct Settings { - pub database: DatabaseSettings, - pub application_port: u16, -} - -#[derive(serde::Deserialize)] -pub struct DatabaseSettings { - pub username: String, - pub password: String, - pub port: u16, - pub host: String, - pub database_name: String, -} - -impl DatabaseSettings { - pub fn connection_string(&self) -> String { - format!( - "postgres://{}:{}@{}:{}/{}", - self.username, self.password, self.host, self.port, self.database_name - ) - } - - pub fn connection_string_without_db(&self) -> String { - format!( - "postgres://{}:{}@{}:{}", - self.username, self.password, self.host, self.port - ) - } -} - -pub fn get_configuration() -> Result { - let mut settings = config::Config::default(); - - settings.merge(config::File::with_name("configuration"))?; - - settings.try_into() -} diff --git a/chapter04/src/main.rs b/chapter04/src/main.rs deleted file mode 100644 index 6d032d2..0000000 --- a/chapter04/src/main.rs +++ /dev/null @@ -1,27 +0,0 @@ -use chapter04::configuration::get_configuration; -use chapter04::startup::run; -use chapter04::telemetry::{get_subscriber, init_subscriber}; -use sqlx::postgres::PgPool; -use std::net::TcpListener; - -#[actix_rt::main] -async fn main() -> std::io::Result<()> { - let subscriber = get_subscriber("zero2prod".into(), "info".into()); - init_subscriber(subscriber); - - let configuration = get_configuration().expect("Failed to read configuration."); - let connection_pool = PgPool::connect(&configuration.database.connection_string()) - .await - .expect("Failed to connect to Postgres."); - - // Here we choose to bind explicitly to localhost, 127.0.0.1, for security - // reasons. This binding may cause issues in some environments. For example, - // it causes connectivity issues running in WSL2, where you cannot reach the - // server when it is bound to WSL2's localhost interface. As a workaround, - // you can choose to bind to all interfaces, 0.0.0.0, instead, but be aware - // of the security implications when you expose the server on all interfaces. - let address = format!("127.0.0.1:{}", configuration.application_port); - let listener = TcpListener::bind(address)?; - run(listener, connection_pool)?.await?; - Ok(()) -} diff --git a/chapter04/src/routes/health_check.rs b/chapter04/src/routes/health_check.rs deleted file mode 100644 index d7eb4e0..0000000 --- a/chapter04/src/routes/health_check.rs +++ /dev/null @@ -1,5 +0,0 @@ -use actix_web::HttpResponse; - -pub async fn health_check() -> HttpResponse { - HttpResponse::Ok().finish() -} diff --git a/chapter04/src/routes/mod.rs b/chapter04/src/routes/mod.rs deleted file mode 100644 index 90ffeed..0000000 --- a/chapter04/src/routes/mod.rs +++ /dev/null @@ -1,5 +0,0 @@ -mod health_check; -mod subscriptions; - -pub use health_check::*; -pub use subscriptions::*; diff --git a/chapter04/tests/health_check.rs b/chapter04/tests/health_check.rs deleted file mode 100644 index c0ac230..0000000 --- a/chapter04/tests/health_check.rs +++ /dev/null @@ -1,143 +0,0 @@ -use chapter04::configuration::{get_configuration, DatabaseSettings}; -use chapter04::startup::run; -use chapter04::telemetry::{get_subscriber, init_subscriber}; -use sqlx::{Connection, Executor, PgConnection, PgPool}; -use std::net::TcpListener; -use uuid::Uuid; - -// Ensure that the `tracing` stack is only initialised once using `lazy_static` -lazy_static::lazy_static! { - static ref TRACING: () = { - let filter = if std::env::var("TEST_LOG").is_ok() { "debug" } else { "" }; - let subscriber = get_subscriber("test".into(), filter.into()); - init_subscriber(subscriber); - }; -} - -pub struct TestApp { - pub address: String, - pub db_pool: PgPool, -} - -async fn spawn_app() -> TestApp { - // The first time `initialize` is invoked the code in `TRACING` is executed. - // All other invocations will instead skip execution. - lazy_static::initialize(&TRACING); - - let listener = TcpListener::bind("127.0.0.1:0").expect("Failed to bind random port"); - // We retrieve the port assigned to us by the OS - let port = listener.local_addr().unwrap().port(); - let address = format!("http://127.0.0.1:{}", port); - - let mut configuration = get_configuration().expect("Failed to read configuration."); - configuration.database.database_name = Uuid::new_v4().to_string(); - let connection_pool = configure_database(&configuration.database).await; - - let server = run(listener, connection_pool.clone()).expect("Failed to bind address"); - let _ = tokio::spawn(server); - TestApp { - address, - db_pool: connection_pool, - } -} - -pub async fn configure_database(config: &DatabaseSettings) -> PgPool { - // Create database - let mut connection = PgConnection::connect(&config.connection_string_without_db()) - .await - .expect("Failed to connect to Postgres"); - connection - .execute(&*format!(r#"CREATE DATABASE "{}";"#, config.database_name)) - .await - .expect("Failed to create database."); - - // Migrate database - let connection_pool = PgPool::connect(&config.connection_string()) - .await - .expect("Failed to connect to Postgres."); - sqlx::migrate!("./migrations") - .run(&connection_pool) - .await - .expect("Failed to migrate the database"); - - connection_pool -} - -#[actix_rt::test] -async fn health_check_works() { - // Arrange - let app = spawn_app().await; - let client = reqwest::Client::new(); - - // Act - let response = client - // Use the returned application address - .get(&format!("{}/health_check", &app.address)) - .send() - .await - .expect("Failed to execute request."); - - // Assert - assert!(response.status().is_success()); - assert_eq!(Some(0), response.content_length()); -} - -#[actix_rt::test] -async fn subscribe_returns_a_200_for_valid_form_data() { - // Arrange - let app = spawn_app().await; - let client = reqwest::Client::new(); - let body = "name=le%20guin&email=ursula_le_guin%40gmail.com"; - - // Act - let response = client - .post(&format!("{}/subscriptions", &app.address)) - .header("Content-Type", "application/x-www-form-urlencoded") - .body(body) - .send() - .await - .expect("Failed to execute request."); - - // Assert - assert_eq!(200, response.status().as_u16()); - - let saved = sqlx::query!("SELECT email, name FROM subscriptions",) - .fetch_one(&app.db_pool) - .await - .expect("Failed to fetch saved subscription."); - - assert_eq!(saved.email, "ursula_le_guin@gmail.com"); - assert_eq!(saved.name, "le guin"); -} - -#[actix_rt::test] -async fn subscribe_returns_a_400_when_data_is_missing() { - // Arrange - let app = spawn_app().await; - let client = reqwest::Client::new(); - let test_cases = vec![ - ("name=le%20guin", "missing the email"), - ("email=ursula_le_guin%40gmail.com", "missing the name"), - ("", "missing both name and email"), - ]; - - for (invalid_body, error_message) in test_cases { - // Act - let response = client - .post(&format!("{}/subscriptions", &app.address)) - .header("Content-Type", "application/x-www-form-urlencoded") - .body(invalid_body) - .send() - .await - .expect("Failed to execute request."); - - // Assert - assert_eq!( - 400, - response.status().as_u16(), - // Additional customised error message on test failure - "The API did not fail with 400 Bad Request when the payload was {}.", - error_message - ); - } -} diff --git a/chapter05/.env b/chapter05/.env deleted file mode 100644 index 88cfb53..0000000 --- a/chapter05/.env +++ /dev/null @@ -1 +0,0 @@ -DATABASE_URL="postgres://postgres:password@localhost:5432/newsletter" diff --git a/chapter05/Cargo.toml b/chapter05/Cargo.toml deleted file mode 100644 index 62cea9a..0000000 --- a/chapter05/Cargo.toml +++ /dev/null @@ -1,34 +0,0 @@ -[package] -name = "chapter05" -version = "0.1.0" -authors = ["LukeMathWalker "] -edition = "2018" - -# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html -[lib] -path = "src/lib.rs" - -[[bin]] -path = "src/main.rs" -name = "chapter05" - -[dependencies] -actix-web = "3.0.0" -actix-rt = "1.1.1" -tokio = "0.2.22" -serde = "1.0.115" -config = { version = "0.10.1", default-features = false, features = ["yaml"] } -sqlx = { version = "0.4.0-beta.1", default-features = false, features = [ "runtime-tokio", "macros", "postgres", "uuid", "chrono", "migrate", "offline"] } -uuid = { version = "0.8.1", features = ["v4"] } -chrono = "0.4.15" -tracing = "0.1.19" -tracing-futures = "0.2.4" -tracing-subscriber = { version = "0.2.12", features = ["registry", "env-filter"] } -tracing-bunyan-formatter = "0.1.6" -tracing-log = "0.1.1" -tracing-actix-web = "0.2.0" -serde-aux = "1.0.1" - -[dev-dependencies] -reqwest = { version = "0.10.7", features = ["json"] } -lazy_static = "1.4.0" diff --git a/chapter05/migrations/20200823135036_create_subscriptions_table.sql b/chapter05/migrations/20200823135036_create_subscriptions_table.sql deleted file mode 100644 index 2c0d262..0000000 --- a/chapter05/migrations/20200823135036_create_subscriptions_table.sql +++ /dev/null @@ -1,8 +0,0 @@ --- Create Subscriptions Table -CREATE TABLE subscriptions( - id uuid NOT NULL, - PRIMARY KEY (id), - email TEXT NOT NULL UNIQUE, - name TEXT NOT NULL, - subscribed_at timestamptz NOT NULL -); \ No newline at end of file diff --git a/chapter05/scripts/init_db.sh b/chapter05/scripts/init_db.sh deleted file mode 100755 index cc1df04..0000000 --- a/chapter05/scripts/init_db.sh +++ /dev/null @@ -1,40 +0,0 @@ -#!/usr/bin/env bash -set -x -set -eo pipefail - -# Check if a custom user has been set, otherwise default to 'postgres' -DB_USER=${POSTGRES_USER:=postgres} -# Check if a custom password has been set, otherwise default to 'password' -DB_PASSWORD="${POSTGRES_PASSWORD:=password}" -# Check if a custom password has been set, otherwise default to 'newsletter' -DB_NAME="${POSTGRES_DB:=newsletter}" -# Check if a custom port has been set, otherwise default to '5432' -DB_PORT="${POSTGRES_PORT:=5432}" - -# Allow to skip Docker if a dockerized Postgres database is already running -if [[ -z "${SKIP_DOCKER}" ]] -then - # Launch postgres using Docker - docker run \ - -e POSTGRES_USER=${DB_USER} \ - -e POSTGRES_PASSWORD=${DB_PASSWORD} \ - -e POSTGRES_DB=${DB_NAME} \ - -p "${DB_PORT}":5432 \ - -d postgres \ - postgres -N 1000 - # ^ Increased maximum number of connections for testing purposes -fi - -# Keep pinging Postgres until it's ready to accept commands -until PGPASSWORD="${DB_PASSWORD}" psql -h "localhost" -U "${DB_USER}" -p "${DB_PORT}" -d "postgres" -c '\q'; do - >&2 echo "Postgres is still unavailable - sleeping" - sleep 1 -done - ->&2 echo "Postgres is up and running on port ${DB_PORT} - running migrations now!" - -export DATABASE_URL=postgres://${DB_USER}:${DB_PASSWORD}@localhost:${DB_PORT}/${DB_NAME} -sqlx database create -sqlx migrate run - ->&2 echo "Postgres has been migrated, ready to go!" diff --git a/chapter05/src/lib.rs b/chapter05/src/lib.rs deleted file mode 100644 index 5d8e21e..0000000 --- a/chapter05/src/lib.rs +++ /dev/null @@ -1,5 +0,0 @@ -#![allow(clippy::toplevel_ref_arg)] -pub mod configuration; -pub mod routes; -pub mod startup; -pub mod telemetry; diff --git a/chapter05/src/routes/health_check.rs b/chapter05/src/routes/health_check.rs deleted file mode 100644 index d7eb4e0..0000000 --- a/chapter05/src/routes/health_check.rs +++ /dev/null @@ -1,5 +0,0 @@ -use actix_web::HttpResponse; - -pub async fn health_check() -> HttpResponse { - HttpResponse::Ok().finish() -} diff --git a/chapter05/src/routes/mod.rs b/chapter05/src/routes/mod.rs deleted file mode 100644 index 90ffeed..0000000 --- a/chapter05/src/routes/mod.rs +++ /dev/null @@ -1,5 +0,0 @@ -mod health_check; -mod subscriptions; - -pub use health_check::*; -pub use subscriptions::*; diff --git a/chapter05/src/routes/subscriptions.rs b/chapter05/src/routes/subscriptions.rs deleted file mode 100644 index fd801af..0000000 --- a/chapter05/src/routes/subscriptions.rs +++ /dev/null @@ -1,52 +0,0 @@ -use actix_web::{web, HttpResponse}; -use chrono::Utc; -use sqlx::PgPool; -use uuid::Uuid; - -#[derive(serde::Deserialize)] -pub struct FormData { - email: String, - name: String, -} - -#[tracing::instrument( - name = "Adding a new subscriber", - skip(form, pool), - fields( - email = %form.email, - name = %form.name - ) -)] -pub async fn subscribe( - form: web::Form, - pool: web::Data, -) -> Result { - insert_subscriber(&pool, &form) - .await - .map_err(|_| HttpResponse::InternalServerError().finish())?; - Ok(HttpResponse::Ok().finish()) -} - -#[tracing::instrument( - name = "Saving new subscriber details in the database", - skip(form, pool) -)] -pub async fn insert_subscriber(pool: &PgPool, form: &FormData) -> Result<(), sqlx::Error> { - sqlx::query!( - r#" - INSERT INTO subscriptions (id, email, name, subscribed_at) - VALUES ($1, $2, $3, $4) - "#, - Uuid::new_v4(), - form.email, - form.name, - Utc::now() - ) - .execute(pool) - .await - .map_err(|e| { - tracing::error!("Failed to execute query: {:?}", e); - e - })?; - Ok(()) -} diff --git a/chapter05/src/startup.rs b/chapter05/src/startup.rs deleted file mode 100644 index d091772..0000000 --- a/chapter05/src/startup.rs +++ /dev/null @@ -1,21 +0,0 @@ -use crate::routes::{health_check, subscribe}; -use actix_web::dev::Server; -use actix_web::web::Data; -use actix_web::{web, App, HttpServer}; -use sqlx::PgPool; -use std::net::TcpListener; -use tracing_actix_web::TracingLogger; - -pub fn run(listener: TcpListener, db_pool: PgPool) -> Result { - let db_pool = Data::new(db_pool); - let server = HttpServer::new(move || { - App::new() - .wrap(TracingLogger) - .route("/health_check", web::get().to(health_check)) - .route("/subscriptions", web::post().to(subscribe)) - .app_data(db_pool.clone()) - }) - .listen(listener)? - .run(); - Ok(server) -} diff --git a/chapter05/src/telemetry.rs b/chapter05/src/telemetry.rs deleted file mode 100644 index 27168fb..0000000 --- a/chapter05/src/telemetry.rs +++ /dev/null @@ -1,29 +0,0 @@ -use tracing::subscriber::set_global_default; -use tracing::Subscriber; -use tracing_bunyan_formatter::{BunyanFormattingLayer, JsonStorageLayer}; -use tracing_log::LogTracer; -use tracing_subscriber::{layer::SubscriberExt, EnvFilter, Registry}; - -/// Compose multiple layers into a `tracing`'s subscriber. -/// -/// # Implementation Notes -/// -/// We are using `impl Subscriber` as return type to avoid having to spell out the actual -/// type of the returned subscriber, which is indeed quite complex. -pub fn get_subscriber(name: String, env_filter: String) -> impl Subscriber + Sync + Send { - let env_filter = - EnvFilter::try_from_default_env().unwrap_or_else(|_| EnvFilter::new(env_filter)); - let formatting_layer = BunyanFormattingLayer::new(name, std::io::stdout); - Registry::default() - .with(env_filter) - .with(JsonStorageLayer) - .with(formatting_layer) -} - -/// Register a subscriber as global default to process span data. -/// -/// It should only be called once! -pub fn init_subscriber(subscriber: impl Subscriber + Sync + Send) { - LogTracer::init().expect("Failed to set logger"); - set_global_default(subscriber).expect("Failed to set subscriber"); -} diff --git a/chapter05/configuration/base.yaml b/configuration/base.yaml similarity index 100% rename from chapter05/configuration/base.yaml rename to configuration/base.yaml diff --git a/chapter05/configuration/local.yaml b/configuration/local.yaml similarity index 100% rename from chapter05/configuration/local.yaml rename to configuration/local.yaml diff --git a/chapter05/configuration/production.yaml b/configuration/production.yaml similarity index 100% rename from chapter05/configuration/production.yaml rename to configuration/production.yaml diff --git a/chapter03-1/migrations/20200823135036_create_subscriptions_table.sql b/migrations/20200823135036_create_subscriptions_table.sql similarity index 100% rename from chapter03-1/migrations/20200823135036_create_subscriptions_table.sql rename to migrations/20200823135036_create_subscriptions_table.sql diff --git a/chapter03-1/scripts/init_db.sh b/scripts/init_db.sh similarity index 100% rename from chapter03-1/scripts/init_db.sh rename to scripts/init_db.sh diff --git a/chapter05/spec.yaml b/spec.yaml similarity index 95% rename from chapter05/spec.yaml rename to spec.yaml index c2c0870..52c15fa 100644 --- a/chapter05/spec.yaml +++ b/spec.yaml @@ -6,8 +6,8 @@ region: fra services: - name: zero2prod # Relative to the repository root - dockerfile_path: chapter05/Dockerfile - source_dir: chapter05 + dockerfile_path: Dockerfile + source_dir: . github: branch: ch-05 deploy_on_push: true diff --git a/chapter05/sqlx-data.json b/sqlx-data.json similarity index 100% rename from chapter05/sqlx-data.json rename to sqlx-data.json diff --git a/chapter05/src/configuration.rs b/src/configuration.rs similarity index 100% rename from chapter05/src/configuration.rs rename to src/configuration.rs diff --git a/chapter04/src/lib.rs b/src/lib.rs similarity index 100% rename from chapter04/src/lib.rs rename to src/lib.rs diff --git a/chapter05/src/main.rs b/src/main.rs similarity index 84% rename from chapter05/src/main.rs rename to src/main.rs index 102e50a..b89442a 100644 --- a/chapter05/src/main.rs +++ b/src/main.rs @@ -1,8 +1,8 @@ -use chapter05::configuration::get_configuration; -use chapter05::startup::run; -use chapter05::telemetry::{get_subscriber, init_subscriber}; use sqlx::postgres::PgPoolOptions; use std::net::TcpListener; +use zero2prod::configuration::get_configuration; +use zero2prod::startup::run; +use zero2prod::telemetry::{get_subscriber, init_subscriber}; #[actix_rt::main] async fn main() -> std::io::Result<()> { diff --git a/chapter03-1/src/routes/health_check.rs b/src/routes/health_check.rs similarity index 100% rename from chapter03-1/src/routes/health_check.rs rename to src/routes/health_check.rs diff --git a/chapter03-1/src/routes/mod.rs b/src/routes/mod.rs similarity index 100% rename from chapter03-1/src/routes/mod.rs rename to src/routes/mod.rs diff --git a/chapter04/src/routes/subscriptions.rs b/src/routes/subscriptions.rs similarity index 100% rename from chapter04/src/routes/subscriptions.rs rename to src/routes/subscriptions.rs diff --git a/chapter04/src/startup.rs b/src/startup.rs similarity index 100% rename from chapter04/src/startup.rs rename to src/startup.rs diff --git a/chapter04/src/telemetry.rs b/src/telemetry.rs similarity index 100% rename from chapter04/src/telemetry.rs rename to src/telemetry.rs diff --git a/chapter05/tests/health_check.rs b/tests/health_check.rs similarity index 96% rename from chapter05/tests/health_check.rs rename to tests/health_check.rs index 11db352..8c5ca8e 100644 --- a/chapter05/tests/health_check.rs +++ b/tests/health_check.rs @@ -1,9 +1,9 @@ -use chapter05::configuration::{get_configuration, DatabaseSettings}; -use chapter05::startup::run; -use chapter05::telemetry::{get_subscriber, init_subscriber}; use sqlx::{Connection, Executor, PgConnection, PgPool}; use std::net::TcpListener; use uuid::Uuid; +use zero2prod::configuration::{get_configuration, DatabaseSettings}; +use zero2prod::startup::run; +use zero2prod::telemetry::{get_subscriber, init_subscriber}; // Ensure that the `tracing` stack is only initialised once using `lazy_static` lazy_static::lazy_static! {