From f52ff61032ac26195d878d70869473df0eacc542 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ya=C4=9Fmur=20Oymak?= <yagmur.oymak@gmail.com>
Date: Thu, 22 Aug 2019 12:28:43 +0300
Subject: [PATCH 1/3] Implement DELETE method

---
 src/lib.rs | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/src/lib.rs b/src/lib.rs
index 20bd9a2..4fc72b1 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -30,6 +30,11 @@ pub fn run(config: Config) -> Result<(), Box<dyn Error>> {
             .service(
                 web::resource("/")
                     .route(web::post().to_async(new_paste))
+                    .wrap(basic_auth.clone()),
+            )
+            .service(
+                web::resource("/{paste_id}")
+                    .route(web::delete().to_async(delete_paste))
                     .wrap(basic_auth),
             )
             .service(send_paste)
@@ -107,6 +112,20 @@ fn new_paste(
     })
 }
 
+fn delete_paste(
+    config: web::Data<Config>,
+    paste_id: web::Path<String>,
+) -> impl Future<Item = HttpResponse, Error = actix_web::Error> {
+    web::block(move || {
+        let file_path = format!("{}/{}", config.paste_dir, paste_id);
+        fs::remove_file(file_path)
+    })
+    .then(|res| match res {
+        Ok(_) => Ok(HttpResponse::Ok().body("")),
+        Err(_) => Ok(HttpResponse::NotFound().into()),
+    })
+}
+
 #[get("/{paste_id}")]
 fn send_paste(
     config: web::Data<Config>,

From b2d2df107807c9137841e376e6fcef9767230120 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ya=C4=9Fmur=20Oymak?= <yagmur.oymak@gmail.com>
Date: Thu, 5 Sep 2019 12:28:47 +0300
Subject: [PATCH 2/3] Update README.md

---
 README.md | 1 -
 1 file changed, 1 deletion(-)

diff --git a/README.md b/README.md
index 76f29b6..0b286dc 100644
--- a/README.md
+++ b/README.md
@@ -40,5 +40,4 @@ base URL to prepend to the `POST` responses will be configured using a config fi
 After implementing a very basic API with `GET` and `POST` requests, I'm planning to add the following features.
 - [x] HTTP basic authentication for posting/deleting pastes.
 - [x] Syntax highlighting with [syntect](https://github.com/trishume/syntect).
-- [ ] `DELETE` method to delete pastes.
 - [ ] Configuration via a config file.

From 870a127839eb941f8ccca8e4f199cc3defdac132 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ya=C4=9Fmur=20Oymak?= <yagmur.oymak@gmail.com>
Date: Thu, 7 Nov 2019 22:02:58 +0300
Subject: [PATCH 3/3] Add test for DELETE method

---
 src/lib.rs | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/src/lib.rs b/src/lib.rs
index 4fc72b1..f2e0c82 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -304,6 +304,35 @@ mod tests {
         assert_eq!(paste_content, file_content);
     }
 
+    #[test]
+    fn delete_paste_file() {
+        let test_dir = TempDir::new().unwrap();
+        let config = make_test_config(test_dir.path().to_str().unwrap());
+
+        let data = web::Data::new(config.clone());
+        let mut app = test::init_service(
+            App::new()
+                .register_data(data)
+                .route("/{paste_id}", web::delete().to_async(delete_paste)),
+        );
+
+        let paste_id = "/testpaste";
+        let paste_path = config.paste_dir + paste_id;
+        {
+            let paste_content = b"nonsense";
+            let mut file = File::create(&paste_path).unwrap();
+            file.write_all(paste_content).unwrap();
+        }
+
+        let req = test::TestRequest::delete().uri(paste_id).to_request();
+        let resp = test::call_service(&mut app, req);
+        assert_eq!(resp.status(), http::StatusCode::OK);
+        assert!(
+            File::open(paste_path).is_err(),
+            "Deleted paste file should not exist"
+        );
+    }
+
     #[test]
     fn auth_valid_creds() {
         let config = make_test_config("unused path");