Skip to content

Commit ae1557d

Browse files
glen-84michaelstaib
authored andcommitted
Fixed parsing issue in Utf8GraphQLRequestParser#ParseStringOrNull (#7703)
1 parent 2ae0bbc commit ae1557d

File tree

2 files changed

+27
-0
lines changed

2 files changed

+27
-0
lines changed

src/HotChocolate/Language/src/Language.Web/Utf8GraphQLRequestParser.Utilities.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ public ref partial struct Utf8GraphQLRequestParser
1212
case TokenKind.String:
1313
if(_reader.Value.Length == 0)
1414
{
15+
_reader.MoveNext();
1516
return null;
1617
}
1718

src/HotChocolate/Language/test/Language.Tests/Parser/GraphQLRequestParserTests.cs

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -661,6 +661,32 @@ public void Parse_Invalid_Query()
661661
});
662662
}
663663

664+
[Fact]
665+
public void Parse_Empty_OperationName()
666+
{
667+
// arrange
668+
var source = Encoding.UTF8.GetBytes(
669+
"""
670+
{
671+
"operationName": "",
672+
"query": "{}"
673+
}
674+
""".NormalizeLineBreaks());
675+
var parserOptions = new ParserOptions();
676+
var requestParser = new Utf8GraphQLRequestParser(
677+
source,
678+
parserOptions,
679+
new DocumentCache(),
680+
new Sha256DocumentHashProvider());
681+
682+
// act
683+
var batch = requestParser.Parse();
684+
685+
// assert
686+
var request = Assert.Single(batch);
687+
Assert.Null(request.OperationName);
688+
}
689+
664690
[Fact]
665691
public void Parse_Empty_Json()
666692
{

0 commit comments

Comments
 (0)