Skip to content

Commit

Permalink
fix: escaped unicode validation
Browse files Browse the repository at this point in the history
  • Loading branch information
tdakkota committed Jan 13, 2022
1 parent ed91401 commit 78e8088
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 13 deletions.
14 changes: 2 additions & 12 deletions dec_skip.go
Original file line number Diff line number Diff line change
Expand Up @@ -302,20 +302,10 @@ var (
'8': 1, '9': 1,

'A': 1, 'B': 1, 'C': 1, 'D': 1,
'E': 1, 'F': 1, 'G': 1, 'H': 1,
'I': 1, 'J': 1, 'K': 1, 'L': 1,
'M': 1, 'N': 1, 'O': 1, 'P': 1,
'Q': 1, 'R': 1, 'S': 1, 'T': 1,
'U': 1, 'V': 1, 'W': 1, 'X': 1,
'Y': 1, 'Z': 1,
'E': 1, 'F': 1,

'a': 1, 'b': 1, 'c': 1, 'd': 1,
'e': 1, 'f': 1, 'g': 1, 'h': 1,
'i': 1, 'j': 1, 'k': 1, 'l': 1,
'm': 1, 'n': 1, 'o': 1, 'p': 1,
'q': 1, 'r': 1, 's': 1, 't': 1,
'u': 1, 'v': 1, 'w': 1, 'x': 1,
'y': 1, 'z': 1,
'e': 1, 'f': 1,
}
)

Expand Down
21 changes: 20 additions & 1 deletion dec_skip_cases_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,12 @@ func TestDecoder_Skip(t *testing.T) {
`""`, // valid
`"hello"`, // valid
`"`, // invalid
`"\`, // invalid
`"\"`, // invalid
`"\u"`, // invalid
`"\u`, // invalid
`"\u1`, // invalid
`"\u12`, // invalid
`"\u123`, // invalid
`"\u\n"`, // invalid
`"\u1\n"`, // invalid
`"\u12\n"`, // invalid
Expand All @@ -49,6 +53,21 @@ func TestDecoder_Skip(t *testing.T) {
"\"\t\"", // invalid
"\"\\b\x06\"", // invalid
`"\t"`, // valid
`"\n"`, // valid
`"\r"`, // valid
`"\b"`, // valid
`"\f"`, // valid
`"\/"`, // valid
`"\\"`, // valid
"\"\\u000X\"", // invalid
"\"\\uxx0X\"", // invalid
"\"\\uxxxx\"", // invalid
"\"\\u000.\"", // invalid
"\"\\u0000\"", // valid
"\"\\ua123\"", // valid
"\"\\uffff\"", // valid
"\"\\ueeee\"", // valid
"\"\\uFFFF\"", // valid
},
})
numberCase := testCase{
Expand Down

0 comments on commit 78e8088

Please sign in to comment.