prevent too long path

sbcgua · Feb 22, 2020 · c52b666 · c52b666
1 parent 34c10f8
commit c52b666
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/lib/params.js b/lib/params.js
@@ -9,6 +9,7 @@ function parsePathParams({pathParameters}) {
     if (!pathParameters.sourcePath) throw Error('Unexpected source path');
     const segments = pathParameters.sourcePath.split('/').filter(s => s !== '');
     if (segments.length === 0) throw Error('Unexpected path');
+    if (segments.length > 20) throw Error('Too many path segments');
 
     const STATES = enumify(['TYPE', 'OWNER', 'REPO', 'FILE', 'ATTR', 'END']);
     let expected = STATES.TYPE;

diff --git a/lib/params.test.js b/lib/params.test.js
@@ -109,6 +109,13 @@ describe('Regular Parser tests', () => {
             },
         })).toThrow('Unexpect path segment');
     });
+    test('should throw on too long path', () => {
+        expect(() => parsePathParams({
+            pathParameters: {
+                sourcePath: new Array(21).fill('a').join('/'),
+            },
+        })).toThrow('Too many path segments');
+    });
 });
 
 describe('apack Parser tests', () => {