astFromValue: reject non-finite numbers (#2437)

Author: IvanGoncharov

src/utilities/__tests__/astFromValue-test.js

@@ -205,6 +205,11 @@ describe('astFromValue', () => {
       value: 'value',
     });
 
+    expect(astFromValue(NaN, passthroughScalar)).to.equal(null);
+    expect(() => astFromValue(Infinity, passthroughScalar)).to.throw(
+      'Cannot convert value to AST: Infinity.',
+    );
+
     const returnNullScalar = new GraphQLScalarType({
       name: 'ReturnNullScalar',
       serialize() {

src/utilities/astFromValue.js

@@ -1,5 +1,6 @@
 // @flow strict
 
+import isFinite from '../polyfills/isFinite';
 import arrayFrom from '../polyfills/arrayFrom';
 import objectValues from '../polyfills/objectValues';
 
@@ -116,7 +117,7 @@ export function astFromValue(value: mixed, type: GraphQLInputType): ?ValueNode {
     }
 
     // JavaScript numbers can be Int or Float values.
-    if (typeof serialized === 'number') {
+    if (typeof serialized === 'number' && isFinite(serialized)) {
       const stringNum = String(serialized);
       return integerStringRegExp.test(stringNum)
         ? { kind: Kind.INT, value: stringNum }