Merge pull request #27 from Sparticuz/feature/replace-input

.eslintrc.js

@@ -1,4 +1,3 @@
-// eslint-disable-next-line node/no-unpublished-require
 require("@rushstack/eslint-patch/modern-module-resolution");
 
 module.exports = {

.github/workflows/node.js.yml

@@ -22,7 +22,7 @@ jobs:
     - name: Install QPDF
       run: |
         sudo apt-get install wget unzip -y
-        wget -O /tmp/qpdf.zip https://github.com/qpdf/qpdf/releases/download/v11.1.0/qpdf-11.1.0-bin-linux-x86_64.zip
+        wget -O /tmp/qpdf.zip https://github.com/qpdf/qpdf/releases/download/v11.2.0/qpdf-11.2.0-bin-linux-x86_64.zip
         sudo unzip -d / /tmp/qpdf.zip
     - uses: actions/checkout@v3
     - name: Use Node.js ${{ matrix.node-version }}

.github/workflows/npm-publish.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: 14
+          node-version: 18
           registry-url: https://registry.npmjs.org/
       - run: npm ci
       - run: npm run build

package.json

@@ -9,7 +9,7 @@
     "build": "tsc -p tsconfig.build.json"
   },
   "engines": {
-    "node": ">=14"
+    "node": ">=14.18.0"
   },
   "repository": {
     "type": "git",
@@ -48,16 +48,16 @@
   "homepage": "https://github.com/Sparticuz/node-qpdf2",
   "devDependencies": {
     "@rushstack/eslint-patch": "^1.2.0",
-    "@sparticuz/eslint-config": "^7.1.8",
+    "@sparticuz/eslint-config": "^7.1.11",
     "@tsconfig/node14": "^1.0.3",
     "@types/node": "^14.18.29",
-    "@typescript-eslint/eslint-plugin": "^5.37.0",
-    "@typescript-eslint/parser": "^5.37.0",
-    "ava": "^4.3.3",
+    "@typescript-eslint/eslint-plugin": "^5.48.0",
+    "@typescript-eslint/parser": "^5.48.0",
+    "ava": "^5.1.0",
     "c8": "^7.12.0",
-    "eslint": "^8.23.1",
-    "prettier": "^2.7.1",
+    "eslint": "^8.31.0",
+    "prettier": "^2.8.1",
     "ts-node": "^10.9.1",
-    "typescript": "^4.8.3"
+    "typescript": "^4.9.4"
   }
 }

src/encrypt.ts

@@ -125,7 +125,12 @@ export const encrypt = async (userPayload: EncryptOptions): Promise<Buffer> => {
   callArguments.push("--", payload.input);
 
   if (payload.output) {
-    callArguments.push(payload.output);
+    // If the input and output locations are the same, and overwrite is true, replace the input file
+    if (payload.input === payload.output && payload.overwrite) {
+      callArguments.push("--replace-input");
+    } else {
+      callArguments.push(payload.output);
+    }
   } else {
     // Print PDF on stdout
     callArguments.push("-");

src/spawn.ts

@@ -1,6 +1,6 @@
 // Ignore errors about unsafe-arguments, this is because data is unknown
 /* eslint-disable @typescript-eslint/no-unsafe-argument */
-import { spawn } from "child_process";
+import { spawn } from "node:child_process";
 
 export default (callArguments: string[]): Promise<Buffer> => {
   return new Promise((resolve, reject) => {

src/utils.ts

@@ -1,4 +1,4 @@
-import { existsSync } from "fs";
+import { existsSync } from "node:fs";
 
 /**
  * hyphenate will take any value that is dromedary case (camelCase with a initial lowercase letter)

test/encrypt.ts

@@ -1,4 +1,5 @@
 import test from "ava";
+import { copyFile } from "node:fs/promises";
 import { encrypt } from "../src/encrypt";
 
 const input = "test/example.pdf";
@@ -112,3 +113,17 @@ test("should throw if restrictions are wrong", async (t) => {
   );
   t.is(error?.message, "Invalid Restrictions");
 });
+
+test("Should encrypt and overwrite the file", async (t) => {
+  await t.notThrowsAsync(async () => {
+    // First, copy example.pdf to output/overwrite.pdf
+    // eslint-disable-next-line sonarjs/no-duplicate-string
+    await copyFile("test/example.pdf", "test/output/overwrite.pdf");
+    await encrypt({
+      input: "test/output/overwrite.pdf",
+      output: "test/output/overwrite.pdf",
+      overwrite: true,
+      password: "1234",
+    });
+  });
+});