Merge pull request #300 from maxbronnikov10/fix/is-serialized-check

fix: Is serialized broken check
steelbrain · Jul 15, 2024 · 41c8666 · 41c8666
2 parents 322c9cd + e27bb9f
commit 41c8666
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/isSerialized.ts b/src/isSerialized.ts
@@ -46,19 +46,16 @@ export default function isSerialized(givenItem: string, strict = false) {
 
   switch (token) {
     case 's':
+    case 'a':
+    case 'O':
       if (strict) {
         if (item.substr(-2, 1) !== '"') {
           return false
         }
       } else if (item.indexOf('"') === -1) {
         return false
       }
-      break
-
-    // or else fall through
 
-    case 'a':
-    case 'O':
       return item.match(new RegExp(`^${token}:[0-9]+:`, 's')) !== null
 
     case 'b':

diff --git a/test/isSerialized-test.ts b/test/isSerialized-test.ts
@@ -4,6 +4,7 @@ import { isSerialized } from '..'
 test('test whether the content is legitimate serialized content', t => {
   t.is(isSerialized('a:1:{i:0;s:12:"add-post_tag";}'), true)
   t.is(isSerialized('d:100010001000100004049513873408;'), true)
+  t.is(isSerialized('s:3276:"{"a":1}";'), true)
   t.is(isSerialized('i:100000000;'), true)
   t.is(isSerialized('d:1.7976931348623157E+308;'), true)
   t.is(isSerialized('something'), false)