fix: #2482 Infinity and NaN serialise to null

ajv-validator · Aug 11, 2024 · d7b6e92 · d7b6e92
1 parent f06766f
commit d7b6e92
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/lib/compile/jtd/serialize.ts b/lib/compile/jtd/serialize.ts
@@ -229,7 +229,11 @@ function serializeString({gen, data}: SerializeCxt): void {
 }
 
 function serializeNumber({gen, data}: SerializeCxt): void {
-  gen.add(N.json, _`"" + ${data}`)
+  gen.if(
+    _`${data} === Infinity || ${data} === -Infinity || Number.isNaN(${data})`,
+    () => gen.add(N.json, _`null`),
+    () => gen.add(N.json, _`"" + ${data}`)
+  )
 }
 
 function serializeRef(cxt: SerializeCxt): void {

diff --git a/spec/jtd-schema.spec.ts b/spec/jtd-schema.spec.ts
@@ -146,6 +146,19 @@ describe("JSON Type Definition", () => {
     }
   })
 
+  describe("serialize special numeric values", () => {
+    const ajv = new _AjvJTD()
+
+    it(`should serialize Infinity to null`, () => {
+      const serialize = ajv.compileSerializer({type: "float64"})
+      assert.deepStrictEqual(JSON.parse(serialize(Infinity)), null)
+    })
+    it(`should serialize NaN to null`, () => {
+      const serialize = ajv.compileSerializer({type: "float64"})
+      assert.deepStrictEqual(JSON.parse(serialize(NaN)), null)
+    })
+  })
+
   describe("parse", () => {
     let ajv: AjvJTD
     before(() => (ajv = new _AjvJTD()))