Merge pull request #130 from C4illin/release-please--branches--main--components--convertx-frontend

chore(deps): update dependency @biomejs/biome to v1.9.1

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.4.1](https://github.com/C4illin/ConvertX/compare/v0.4.0...v0.4.1) (2024-09-15)
+
+
+### Bug Fixes
+
+* allow non lowercase true and false values, fixes [#122](https://github.com/C4illin/ConvertX/issues/122) ([bef1710](https://github.com/C4illin/ConvertX/commit/bef1710e3376baa7e25c107ded20a40d18b8c6b0))
+
 ## [0.4.0](https://github.com/C4illin/ConvertX/compare/v0.3.3...v0.4.0) (2024-08-26)
 
 

Dockerfile

@@ -1,4 +1,4 @@
-FROM oven/bun:1.1.26-alpine AS base
+FROM oven/bun:1.1.27-alpine AS base
 LABEL org.opencontainers.image.source="https://github.com/C4illin/ConvertX"
 WORKDIR /app
 

README.md

@@ -54,7 +54,7 @@ services:
 or
 
 ```bash
-docker run ghcr.io/c4illin/convertx -p 3000:3000 -v ./data:/app/data
+docker run -p 3000:3000 -v ./data:/app/data ghcr.io/c4illin/convertx
 ```
 
 Then visit `http://localhost:3000` in your browser and create your account. Don't leave it unconfigured and open, as anyone can register the first account.
@@ -76,12 +76,12 @@ Pull requests are welcome! See below and open issues for the list of todos.
 
 ## Todo
 - [x] Add messages for errors in converters
+- [x] Add searchable list of formats
 - [ ] Add options for converters
-- [ ] Add more converters
 - [ ] Divide index.tsx into smaller components
 - [ ] Add tests
-- [ ] Add searchable list of formats
 - [ ] Make the upload button nicer and more easy to drop files on. Support copy paste as well if possible.
+- [ ] Make errors logs visible from the web ui
 - [ ] Add more converters:
   - [ ] [deark](https://github.com/jsummers/deark)
   - [ ] LibreOffice

package.json

@@ -1,6 +1,6 @@
 {
   "name": "convertx-frontend",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "scripts": {
     "dev": "bun run --watch src/index.tsx",
     "hot": "bun run --hot src/index.tsx",
@@ -14,9 +14,9 @@
   "dependencies": {
     "@elysiajs/cookie": "^0.8.0",
     "@elysiajs/html": "1.0.2",
-    "@elysiajs/jwt": "^1.1.0",
+    "@elysiajs/jwt": "^1.1.1",
     "@elysiajs/static": "1.0.3",
-    "elysia": "^1.1.7"
+    "elysia": "^1.1.12"
   },
   "module": "src/index.tsx",
   "type": "module",
@@ -24,32 +24,32 @@
     "start": "bun run src/index.tsx"
   },
   "devDependencies": {
-    "@biomejs/biome": "1.8.3",
+    "@biomejs/biome": "1.9.1",
     "@eslint/compat": "^1.1.1",
-    "@eslint/js": "^9.9.0",
+    "@eslint/js": "^9.9.1",
     "@ianvs/prettier-plugin-sort-imports": "^4.3.1",
     "@kitajs/ts-html-plugin": "^4.0.2",
     "@picocss/pico": "^2.0.6",
-    "@total-typescript/ts-reset": "^0.6.0",
-    "@types/bun": "^1.1.6",
-    "@types/eslint": "^9.6.0",
-    "@types/node": "^22.5.0",
-    "@typescript-eslint/eslint-plugin": "^8.2.0",
-    "@typescript-eslint/parser": "^8.2.0",
+    "@total-typescript/ts-reset": "^0.6.1",
+    "@types/bun": "^1.1.8",
+    "@types/eslint": "^9.6.1",
+    "@types/node": "^22.5.4",
+    "@typescript-eslint/eslint-plugin": "^8.4.0",
+    "@typescript-eslint/parser": "^8.4.0",
     "cpy-cli": "^5.0.0",
-    "eslint": "^9.9.0",
+    "eslint": "^9.9.1",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-deprecation": "^3.0.0",
     "eslint-plugin-eslint-comments": "^3.2.0",
-    "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-isaacscript": "^3.12.2",
+    "eslint-plugin-import": "^2.30.0",
+    "eslint-plugin-isaacscript": "^4.0.0",
     "eslint-plugin-prettier": "^5.2.1",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "knip": "^5.27.3",
+    "knip": "^5.29.2",
     "npm-run-all2": "^6.2.2",
     "prettier": "^3.3.3",
     "typescript": "^5.5.4",
-    "typescript-eslint": "^8.2.0"
+    "typescript-eslint": "^8.4.0"
   },
   "trustedDependencies": [
     "@biomejs/biome"

src/converters/resvg.ts

@@ -9,7 +9,6 @@ export const properties = {
   },
 };
 
-
 export function convert(
   filePath: string,
   fileType: string,
@@ -19,23 +18,20 @@ export function convert(
   options?: any,
 ): Promise<string> {
   return new Promise((resolve, reject) => {
-    exec(
-      `resvg "${filePath}" "${targetPath}"`,
-      (error, stdout, stderr) => {
-        if (error) {
-          reject(`error: ${error}`);
-        }
+    exec(`resvg "${filePath}" "${targetPath}"`, (error, stdout, stderr) => {
+      if (error) {
+        reject(`error: ${error}`);
+      }
 
-        if (stdout) {
-          console.log(`stdout: ${stdout}`);
-        }
+      if (stdout) {
+        console.log(`stdout: ${stdout}`);
+      }
 
-        if (stderr) {
-          console.error(`stderr: ${stderr}`);
-        }
+      if (stderr) {
+        console.error(`stderr: ${stderr}`);
+      }
 
-        resolve("success");
-      },
-    );
+      resolve("success");
+    });
   });
-}
+}

src/index.tsx

@@ -27,11 +27,12 @@ const uploadsDir = "./data/uploads/";
 const outputDir = "./data/output/";
 
 const ACCOUNT_REGISTRATION =
-  process.env.ACCOUNT_REGISTRATION === "true" || false;
+  process.env.ACCOUNT_REGISTRATION?.toLowerCase() === "true" || false;
 
-const HTTP_ALLOWED = process.env.HTTP_ALLOWED === "true" || false;
+const HTTP_ALLOWED =
+  process.env.HTTP_ALLOWED?.toLowerCase() === "true" || false;
 const ALLOW_UNAUTHENTICATED =
-  process.env.ALLOW_UNAUTHENTICATED === "true" || false;
+  process.env.ALLOW_UNAUTHENTICATED?.toLowerCase() === "true" || false;
 
 // fileNames: fileNames,
 // filesToConvert: fileNames.length,