add last used fieldi and some logging

Bump mio from 0.8.8 to 0.8.11

.github/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,38 @@
+---
+name: Bug report
+about: Create a report to help us improve
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Describe the bug**
+A clear and concise description of what the bug is.
+
+**To Reproduce**
+Steps to reproduce the behavior:
+1. Go to '...'
+2. Click on '....'
+3. Scroll down to '....'
+4. See error
+
+**Expected behavior**
+A clear and concise description of what you expected to happen.
+
+**Screenshots**
+If applicable, add screenshots to help explain your problem.
+
+**Desktop (please complete the following information):**
+ - OS: [e.g. iOS]
+ - Browser [e.g. chrome, safari]
+ - Version [e.g. 22]
+
+**Smartphone (please complete the following information):**
+ - Device: [e.g. iPhone6]
+ - OS: [e.g. iOS8.1]
+ - Browser [e.g. stock browser, safari]
+ - Version [e.g. 22]
+
+**Additional context**
+Add any other context about the problem here.

.github/ISSUE_TEMPLATE/feature_request.md

@@ -0,0 +1,20 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Is your feature request related to a problem? Please describe.**
+A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+
+**Describe the solution you'd like**
+A clear and concise description of what you want to happen.
+
+**Describe alternatives you've considered**
+A clear and concise description of any alternative solutions or features you've considered.
+
+**Additional context**
+Add any other context or screenshots about the feature request here.

Cargo.toml

@@ -1,16 +1,17 @@
 [package]
 name = "ics-proxy"
-version = "0.1.0"
+version = "0.1.2"
-edition = "2018"
+edition = "2021"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [dependencies]
-actix-web = "3"
+actix-web = "4.0"
-uuid = { version = "0.8.2", features = ["v4"] }
+uuid = { version = "1.0", features = ["v4"] }
 url = "2.2"
 reqwest = { version = "0.11", features = ["blocking"] }
-tera = "1.12.1"
+tera = "1.15"
 dotenv = "0.15"
 
-sqlx = { version = "0.4.2", features = [ "sqlite", "runtime-actix-native-tls" ] }
+sqlx = { version = "0.5", features = ["sqlite", "runtime-actix-rustls"] }
+chrono = "0.4.38"

Dockerfile

@@ -1,11 +1,13 @@
-FROM rust:1.54 as builder
+FROM rust:1.77 as builder
 RUN apt-get update && apt-get install -y sqlite3 && rm -rf /var/lib/apt/lists/*
+RUN cargo install sqlx-cli
 WORKDIR /usr/src/ics-proxy
 COPY . .
-RUN cd db && ./create_db.sh
+RUN sqlx database create
+RUN sqlx migrate run
 RUN cargo install --path .
 
-FROM debian:buster-slim
+FROM debian:stable-slim
 RUN apt-get update && apt-get install -y sqlite3 openssl ca-certificates && rm -rf /var/lib/apt/lists/*
 WORKDIR app
 COPY --from=builder /usr/local/cargo/bin/ics-proxy ./ics-proxy

README.md

@@ -1,6 +1,13 @@
 # ics-proxy
 
-You can find a running instance of this code on [ics-proxy.de](https://ics-proxy.de). The docker image is published on [DockerHub](https://hub.docker.com/repository/docker/13hannes11/ics-proxy).
+You can find a running instance of this code on [ics-proxy.de](https://ics-proxy.de). The docker image is published on the [GitHub Container Registry](https://github.com/13hannes11/ics-proxy/pkgs/container/ics-proxy).
+
+## Screenshots
+
+![image](https://user-images.githubusercontent.com/9381167/136559243-2a7c9062-33e3-436e-a781-fef3173e1671.png)
+![image](https://user-images.githubusercontent.com/9381167/136559368-3404a94f-35d1-4235-8c98-2f837b75fda0.png)
+
+
 
 ## Motivation
 
@@ -12,6 +19,11 @@ Additionally, each semester this needed to be done as well. I therefore created
 
 The easiest way to build this repository is to use docker. You can simply run `docker build -t ics-proxy .`
 
+Alternatively you can build and run a local dev version with `docker-compose -f docker-compose.dev.yml up`.
+
 ## Deployment
 
 To deploy you can simply use the `docker-compose.yml` file.
+
+## Contributing
+Pull requests are welcome. For major changes, please open an issue first to discuss what you would like to change.

docker-compose.dev.yml

@@ -0,0 +1,12 @@
+version: '3'
+services:
+  ics-proxy:
+    container_name: ics-proxy
+    build: .
+    env_file: .env
+    ports:
+     - "8080:8080"
+    volumes:
+     - database:/app/db
+volumes:
+  database:

migrations/20240425185919_inital_db_scheme.sql

@@ -1,4 +1,5 @@
+-- Add migration script here
 CREATE TABLE IF NOT EXISTS LINKS(
     UUID TEXT NOT NULL PRIMARY KEY,
     DESTINATION TEXT NOT NULL
-);
+);

migrations/20240425193610_add_last_used_column.sql

@@ -0,0 +1,2 @@
+-- Add migration script here
+ALTER TABLE LINKS ADD COLUMN last_used TEXT;

src/main.rs

@@ -2,6 +2,7 @@ use std::collections::HashMap;
 use url::Url;
 
 use actix_web::http::StatusCode;
+use actix_web::web::Data;
 use actix_web::{error, web, App, Error, HttpRequest, HttpResponse, HttpServer, Responder, Result};
 use sqlx::{Pool, Sqlite, SqlitePool};
 use tera::Tera;
@@ -22,11 +23,11 @@ struct Config {
 
 async fn make_ics_request(req: HttpRequest, db_pool: web::Data<Pool<Sqlite>>) -> impl Responder {
     let id = req.match_info().get("id").unwrap_or("");
-
+    println!("serving ics request");
     match Uuid::parse_str(id) {
         Ok(uuid) => match Link::find_by_uuid(uuid.to_string(), db_pool).await {
-            Ok(link) => match reqwest::blocking::get(link.destination) {
+            Ok(link) => match reqwest::get(link.destination).await {
-                Ok(r) => match r.text() {
+                Ok(r) => match r.text().await {
                     Ok(res) => HttpResponse::Ok().content_type("text/calendar").body(res),
                     Err(err) => HttpResponse::Ok()
                         .status(StatusCode::INTERNAL_SERVER_ERROR)
@@ -59,6 +60,7 @@ async fn edit_page(
     db_pool: web::Data<Pool<Sqlite>>,
     conf: web::Data<Config>,
 ) -> Result<HttpResponse, Error> {
+    println!("serving edit page");
     // one uuid: 9228c1a4-8956-4f1c-8b5f-53cc575bd78
     if let Some(uuid_str) = query.get("uuid") {
         match Uuid::parse_str(uuid_str) {
@@ -256,7 +258,7 @@ async fn index_process(
 // store tera template in application state
 async fn index(tmpl: web::Data<tera::Tera>) -> Result<HttpResponse, Error> {
     // TODO: add option to prefill link with parameter
-
+    println!("serving index page");
     let s = tmpl
         .render("index.html", &tera::Context::new())
         .map_err(|_| error::ErrorInternalServerError("Template error"))?;
@@ -291,6 +293,8 @@ async fn main() -> std::io::Result<()> {
         .await
         .expect("could not create db pool");
 
+    sqlx::migrate!("./migrations").run(&db_pool).await.unwrap();
+
     println!(
         "Listening on: {}://{}, open browser and visit have a try!",
         protocol, base_url
@@ -299,9 +303,9 @@ async fn main() -> std::io::Result<()> {
         let tera = Tera::new("templates/**/*.html").unwrap();
 
         App::new()
-            .data(db_pool.clone()) // pass database pool to application so we can access it inside handlers
+            .app_data(Data::new(db_pool.clone())) // pass database pool to application so we can access it inside handlers
-            .data(tera)
+            .app_data(Data::new(tera))
-            .data(conf.clone())
+            .app_data(Data::new(conf.clone()))
             .route("/{id}/events.ics", web::get().to(make_ics_request))
             .service(web::resource("/").route(web::get().to(index)))
             .service(web::resource("/edit").route(web::get().to(edit_page)))

src/model.rs

@@ -1,4 +1,8 @@
 use actix_web::web;
+use chrono::DateTime;
+use chrono::Utc;
+use std::time::SystemTime;
+
 use sqlx::{Pool, Sqlite};
 
 // Change to strings if to much headache
@@ -13,6 +17,7 @@ impl Link {
         pool: web::Data<Pool<Sqlite>>,
     ) -> Result<Link, sqlx::Error> {
         let mut tx = pool.begin().await?;
+        println!("find by uuid {uuid}");
         let rec = sqlx::query!(
             r#"
                     SELECT * FROM links WHERE uuid = $1
@@ -22,6 +27,19 @@ impl Link {
         .fetch_one(&mut tx)
         .await?;
 
+        let now = SystemTime::now();
+        let now: DateTime<Utc> = now.into();
+        let now = now.to_rfc3339();
+
+        sqlx::query!(
+            r#" UPDATE links SET last_used = $1 WHERE uuid = $2"#,
+            now,
+            uuid,
+        )
+        .execute(&mut tx)
+        .await?;
+        tx.commit().await?;
+
         Ok(Link {
             uuid: rec.UUID,
             destination: rec.DESTINATION,
@@ -35,6 +53,7 @@ impl Link {
             .execute(&mut tx)
             .await?;
 
+        println!("update uuid {}", link.uuid);
         tx.commit().await?;
         Ok(link)
     }
@@ -46,6 +65,7 @@ impl Link {
             .execute(&mut tx)
             .await?;
 
+        println!("create uuid {}", link.uuid);
         tx.commit().await?;
         Ok(link)
     }