fix: handle error cases when adding invalid ocafile

Cargo.toml

@@ -22,7 +22,7 @@ dyn-clonable = "0.9.0"
 indexmap = { version = "1.9.3", features = ["serde"] }
 isolang = "2.3.0"
 oca-parser-xls = { version = "2.0.0-rc.2", optional = true }
-oca-rs = "0.3.0-rc.19"
+oca-rs = "0.3.3"
 rusqlite = "0.29.0"
 serde = { version = "1.0", features = ["derive"] }
 serde-value = "0.7.0"

src/routes/oca_bundles.rs

@@ -8,7 +8,21 @@ pub async fn add_oca_file(
     item: web::Bytes,
     _req: HttpRequest,
 ) -> HttpResponse {
-    let ocafile = String::from_utf8(item.to_vec()).unwrap();
+    let ocafile = match String::from_utf8(item.to_vec()) {
+        Ok(parsed) => parsed,
+        Err(e) => {
+            return HttpResponse::UnprocessableEntity()
+                .content_type(ContentType::json())
+                .body(serde_json::to_string(&vec![e.to_string()]).unwrap())
+        }
+    };
+
+    if ocafile.is_empty() {
+        let error = "OCA File can't be empty";
+        return HttpResponse::UnprocessableEntity()
+            .content_type(ContentType::json())
+            .body(serde_json::to_string(&vec![error]).unwrap())
+    }
 
     let result = match oca_facade.lock().unwrap().build_from_ocafile(ocafile) {
         Ok(oca_bundle) => {