@@ -50,7 +50,11 @@ private GraphqlErrorBuilder() {
     }
 
     public GraphqlErrorBuilder message(String message, Object... formatArgs) {
-        this.message = String.format(assertNotNull(message), formatArgs);
+        if (formatArgs == null || formatArgs.length == 0) {
+            this.message = assertNotNull(message);
+        } else {
+            this.message = String.format(assertNotNull(message), formatArgs);
+        }
         return this;
     }
 
 
@@ -76,4 +76,38 @@ class GraphqlErrorBuilderTest extends Specification {
         graphQLError.getPath() == ["field"]
         graphQLError.getLocations() == [new SourceLocation(1, 3)]
     }
+
+    def "java string format is safe"() {
+        when:
+        def gqlErr = GraphqlErrorBuilder.newError().message("This has %s in it").build()
+        then:
+        gqlErr.getMessage() == "This has %s in it"
+
+        when:
+        gqlErr = GraphqlErrorBuilder.newError().message("This has %s in it", null).build()
+        then:
+        gqlErr.getMessage() == "This has %s in it"
+
+        when:
+        gqlErr = GraphqlErrorBuilder.newError().message("This has %s in it", new Object[0]).build()
+        then:
+        gqlErr.getMessage() == "This has %s in it"
+
+        when:
+        gqlErr = GraphqlErrorBuilder.newError().message("This has %s in it", "data").build()
+        then:
+        gqlErr.getMessage() == "This has data in it"
+    }
+
+    def "null message is not acceptable"() {
+        when:
+        GraphqlErrorBuilder.newError().message(null, "a", "b").build()
+        then:
+        thrown(AssertException)
+
+        when:
+        GraphqlErrorBuilder.newError().message(null).build()
+        then:
+        thrown(AssertException)
+    }
 }
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,11 @@ private GraphqlErrorBuilder() {`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`public GraphqlErrorBuilder message(String message, Object... formatArgs) {`
`53`		`- this.message = String.format(assertNotNull(message), formatArgs);`
	`53`	`+ if (formatArgs == null \|\| formatArgs.length == 0) {`
	`54`	`+ this.message = assertNotNull(message);`
	`55`	`+ } else {`
	`56`	`+ this.message = String.format(assertNotNull(message), formatArgs);`
	`57`	`+ }`
`54`	`58`	`return this;`
`55`	`59`	`}`
`56`	`60`