Cargo.lock

@@ -1778,7 +1778,7 @@ dependencies = [
 
 [[package]]
 name = "transbeam"
-version = "0.4.0"
+version = "0.3.0"
 dependencies = [
  "actix",
  "actix-files",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "transbeam"
-version = "0.4.0"
+version = "0.3.0"
 authors = ["xenofem <xenofem@xeno.science>"]
 edition = "2021"
 license = "MIT"

cli/transbeam-cli

@@ -51,6 +51,10 @@ async def send(paths, host, password, lifetime, collection_name=None):
         loader = file_loader([(paths[i], fileMetadata[i]["size"]) for i in range(len(paths))])
         for data in loader:
             await ws.send(data)
+            resp = await ws.recv()
+            if resp != "ack":
+                tqdm.write("unexpected response: {}".format(resp))
+                exit(1)
 
 parser = argparse.ArgumentParser(description="Upload files to transbeam")
 parser.add_argument("-l", "--lifetime", type=int, default=7, help="Lifetime in days for files (default 7)")

src/main.rs

@@ -4,13 +4,13 @@ mod store;
 mod upload;
 mod zip;
 
-use std::{fmt::Debug, ops::Deref, path::PathBuf, str::FromStr};
+use std::{fmt::Debug, path::PathBuf, str::FromStr, ops::Deref};
 
 use actix_http::StatusCode;
-use actix_session::{storage::CookieSessionStore, Session, SessionMiddleware};
+use actix_session::{SessionMiddleware, storage::CookieSessionStore, Session};
 use actix_web::{
-    cookie, error::InternalError, get, middleware::Logger, post, web, App, HttpRequest,
+    error::InternalError, get, middleware::Logger, post, web, App, HttpRequest, HttpResponse,
-    HttpResponse, HttpServer, Responder,
+    HttpServer, Responder, cookie,
 };
 use actix_web_actors::ws;
 use argon2::{Argon2, PasswordVerifier};
@@ -19,7 +19,7 @@ use bytesize::ByteSize;
 use log::{error, warn};
 use password_hash::PasswordHashString;
 use serde::{Deserialize, Serialize};
-use state::{prelude::SizedFile, StateDb};
+use state::{StateDb, prelude::SizedFile};
 use store::{StoredFile, StoredFiles};
 use tokio::fs::File;
 
@@ -93,24 +93,19 @@ struct AdminPage<'a> {
 }
 
 #[get("/admin")]
-async fn admin_panel(
+async fn admin_panel(data: web::Data<AppData>, session: Session) -> actix_web::Result<HttpResponse> {
-    data: web::Data<AppData>,
-    session: Session,
-) -> actix_web::Result<HttpResponse> {
     if let Some(true) = session.get::<bool>("admin")? {
         Ok(AdminPage {
             cachebuster: data.config.cachebuster.clone(),
             base_url: data.config.base_url.clone(),
             stored_files: data.state.read().await.deref(),
-        }
+        }.to_response())
-        .to_response())
     } else {
         Ok(SignedOutAdminPage {
             cachebuster: data.config.cachebuster.clone(),
             base_url: data.config.base_url.clone(),
             incorrect_password: false,
-        }
+        }.to_response())
-        .to_response())
     }
 }
 
@@ -120,26 +115,14 @@ struct AdminPanelSignin {
 }
 
 #[post("/admin")]
-async fn admin_signin(
+async fn admin_signin(req: HttpRequest, data: web::Data<AppData>, form: web::Form<AdminPanelSignin>, session: Session) -> actix_web::Result<HttpResponse> {
-    req: HttpRequest,
+    if Argon2::default().verify_password(form.password.as_bytes(), &data.config.admin_password_hash.password_hash()).is_ok() {
-    data: web::Data<AppData>,
-    form: web::Form<AdminPanelSignin>,
-    session: Session,
-) -> actix_web::Result<HttpResponse> {
-    if Argon2::default()
-        .verify_password(
-            form.password.as_bytes(),
-            &data.config.admin_password_hash.password_hash(),
-        )
-        .is_ok()
-    {
         session.insert("admin", true)?;
         Ok(AdminPage {
             cachebuster: data.config.cachebuster.clone(),
             base_url: data.config.base_url.clone(),
             stored_files: data.state.read().await.deref(),
-        }
+        }.to_response())
-        .to_response())
     } else {
         let ip_addr = get_ip_addr(&req, data.config.reverse_proxy);
         log_auth_failure(&ip_addr);
@@ -147,8 +130,7 @@ async fn admin_signin(
             cachebuster: data.config.cachebuster.clone(),
             base_url: data.config.base_url.clone(),
             incorrect_password: true,
-        }
+        }.to_response();
-        .to_response();
         *resp.status_mut() = StatusCode::FORBIDDEN;
 
         Err(InternalError::from_response("Incorrect password", resp).into())
@@ -393,13 +375,12 @@ async fn main() -> std::io::Result<()> {
     let admin_password_hash: PasswordHashString = env_or_panic("TRANSBEAM_ADMIN_PASSWORD_HASH");
 
     let cookie_secret_base64: String = env_or_panic("TRANSBEAM_COOKIE_SECRET");
-    let cookie_key =
+    let cookie_key = cookie::Key::from(
-        cookie::Key::from(&base64::decode(&cookie_secret_base64).unwrap_or_else(|_| {
+        &base64::decode(&cookie_secret_base64)
-            panic!(
+            .unwrap_or_else(
-                "Value {} for TRANSBEAM_COOKIE_SECRET is not valid base64",
+                |_| panic!("Value {} for TRANSBEAM_COOKIE_SECRET is not valid base64", cookie_secret_base64)
-                cookie_secret_base64
             )
-        }));
+    );
 
     let state_file: PathBuf = match std::env::var("TRANSBEAM_STATE_FILE") {
         Ok(v) => v
@@ -443,10 +424,7 @@ async fn main() -> std::io::Result<()> {
             } else {
                 Logger::default()
             })
-            .wrap(SessionMiddleware::new(
+            .wrap(SessionMiddleware::new(CookieSessionStore::default(), cookie_key.clone()))
-                CookieSessionStore::default(),
-                cookie_key.clone(),
-            ))
             .service(index)
             .service(handle_download)
             .service(download_info)

src/state.rs

@@ -13,11 +13,12 @@ pub mod prelude {
         fn size(&self) -> u64;
 
         fn formatted_size(&self) -> String {
-            bytesize::to_string(self.size(), false).replace(' ', "")
+            bytesize::to_string(self.size(), false).replace(" ", "")
         }
     }
 }
 
+
 mod v0;
 
 pub mod v1 {
@@ -51,9 +52,7 @@ pub mod v1 {
         }
     }
     impl SizedFile for UploadedFile {
-        fn size(&self) -> u64 {
+        fn size(&self) -> u64 { self.size }
-            self.size
-        }
     }
 
     #[derive(Debug, Clone, Deserialize, Serialize)]
@@ -84,9 +83,7 @@ pub mod v1 {
         pub contents: Option<FileSet>,
     }
     impl SizedFile for StoredFile {
-        fn size(&self) -> u64 {
+        fn size(&self) -> u64 { self.size }
-            self.size
-        }
     }
 
     #[derive(Debug, Clone, Deserialize, Serialize)]

src/store.rs

@@ -36,7 +36,7 @@ async fn is_valid_entry(key: &str, info: &StoredFile, storage_dir: &Path) -> boo
         return false;
     }
 
-    let file = if let Ok(f) = File::open(storage_dir.join(key)).await {
+    let file = if let Ok(f) = File::open(storage_dir.join(&key)).await {
         f
     } else {
         error!(
@@ -108,11 +108,7 @@ impl crate::AppData {
     /// Attempts to add a file to the store. Returns an I/O error if
     /// something's broken, or a u64 of the maximum allowed file size
     /// if the file was too big, or a unit if everything worked.
-    pub async fn add_file(
+    pub async fn add_file(&self, key: String, entry: StoredFileWithPassword) -> Result<(), FileAddError> {
-        &self,
-        key: String,
-        entry: StoredFileWithPassword,
-    ) -> Result<(), FileAddError> {
         let mut store = self.state.write().await;
         if store.full(self.config.max_storage_size) {
             return Err(FileAddError::Full);

src/upload.rs

@@ -211,6 +211,10 @@ impl StreamHandler<Result<ws::Message, ws::ProtocolError>> for Uploader {
     }
 }
 
+fn ack(ctx: &mut Context) {
+    ctx.text("ack");
+}
+
 impl Uploader {
     fn notify_error_and_cleanup(&mut self, e: Error, ctx: &mut Context) {
         error!("{}", e);
@@ -346,10 +350,13 @@ impl Uploader {
                     }),
                 );
             }
-            ws::Message::Binary(data)
+            ws::Message::Binary(data) | ws::Message::Continuation(Item::Last(data)) => {
-            | ws::Message::Continuation(Item::FirstBinary(data))
+                let result = self.handle_data(data)?;
-            | ws::Message::Continuation(Item::Continue(data))
+                ack(ctx);
-            | ws::Message::Continuation(Item::Last(data)) => {
+                return Ok(result);
+            }
+            ws::Message::Continuation(Item::FirstBinary(data))
+            | ws::Message::Continuation(Item::Continue(data)) => {
                 return self.handle_data(data);
             }
             ws::Message::Close(reason) => {

static/js/upload.js

@@ -2,8 +2,6 @@ const FILE_CHUNK_SIZE = 16384;
 const MAX_FILES = 256;
 const SAMPLE_WINDOW = 100;
 const STALL_THRESHOLD = 1000;
-const MAX_WS_BUFFER = 1048576;
-const WS_BUFFER_DELAY = 10;
 
 let files = [];
 
@@ -206,11 +204,7 @@ function sendManifest() {
 }
 
 function handleMessage(msg) {
-    if (bytesSent > 0) {
+    if (bytesSent === 0) {
-        console.warn('Received unexpected message from server during upload', msg.data);
-        return;
-    }
-
         let reply;
         try {
             reply = JSON.parse(msg.data);
@@ -243,6 +237,15 @@ function handleMessage(msg) {
         } else if (reply.type === 'error') {
             displayError(reply.details);
         }
+    } else {
+        if (msg.data === 'ack') {
+            sendData();
+        } else {
+            console.error('Received unexpected message from server instead of ack', msg.data);
+            displayError();
+            socket.close();
+        }
+    }
 }
 
 function updateMaxLifetime(lifetime) {
@@ -259,21 +262,10 @@ function updateMaxLifetime(lifetime) {
 }
 
 function sendData() {
-    if (socket.readyState !== 1) {
-        return;
-    }
-
-    while (true) {
     if (fileIndex >= files.length) {
         finishSending();
         return;
     }
-
-        if (socket.bufferedAmount >= MAX_WS_BUFFER) {
-            setTimeout(sendData, WS_BUFFER_DELAY);
-            return;
-        }
-
     const currentFile = files[fileIndex];
     if (byteIndex < currentFile.size) {
         const endpoint = Math.min(byteIndex+FILE_CHUNK_SIZE, currentFile.size);
@@ -295,7 +287,7 @@ function sendData() {
     } else {
         fileIndex += 1;
         byteIndex = 0;
-        }
+        sendData();
     }
 }