Validate filename length (#20)

Shpota · web-flow · commit 8ad025b18d69 · 2023-01-10T17:21:43.000+03:00
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "podil",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "description": "Lightweight and secure database migration tool",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",
diff --git a/src/Podil.ts b/src/Podil.ts
@@ -24,6 +24,9 @@ export class Podil {
             `File '${file}' cannot be executed. Make sure the directory with migrations contains only '.sql' files.`
           )
         }
+        if (file.length > 255) {
+          throw new Error(`Script filename exceeds the allowed 255 symbols limit: ${file}`)
+        }
       }
       const fsScripts = files.sort((a, b) => a.localeCompare(b))
       if (fsScripts.length < appliedScripts.length) {

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "podil",`
`3`		`- "version": "0.3.2",`
	`3`	`+ "version": "0.3.3",`
`4`	`4`	`"description": "Lightweight and secure database migration tool",`
`5`	`5`	`"main": "dist/cjs/index.js",`
`6`	`6`	`"module": "dist/esm/index.js",`
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,9 @@ export class Podil {`
`24`	`24`	`File '${file}' cannot be executed. Make sure the directory with migrations contains only '.sql' files.`
`25`	`25`	`)`
`26`	`26`	`}`
	`27`	`+ if (file.length > 255) {`
	`28`	+ throw new Error(`Script filename exceeds the allowed 255 symbols limit: ${file}`)
	`29`	`+ }`
`27`	`30`	`}`
`28`	`31`	`const fsScripts = files.sort((a, b) => a.localeCompare(b))`
`29`	`32`	`if (fsScripts.length < appliedScripts.length) {`