GlowstoneMC / Glowstone

A fast, customizable and compatible open source server for Minecraft: Java Edition
https://glowstone.net
Other
1.88k stars 270 forks source link

Mojangson.parseCompound not throwing error with invalid syntax #1055

Open clabe45 opened 4 years ago

clabe45 commented 4 years ago

Title: Mojangson.parseCompound not throwing error with invalid syntax

Glowstone build: (commit) 760bff1c4f4fc19ea8507d69924bb7cacb6a9d80

Mojangson.parseCompound doesn't throw a MojangsonParseException with input "{sfasdjflsdfva;rsjtewlq". In singleplayer 1.12, I get the following error when I try to use that as mojangson, so it isn't valid:

Data tag parsing failed: Expected ':' but got ';' at: {sfasdjflsdfva;<--[HERE]

Similarly, the following calls to parseCompound return the following values:

Input Output
{ {}
{" {}
{{} {}
& (error)

It looks like, as long as it begins with a {, no error will be thrown, regardless of the rest of the input.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/80824207-mojangson-parsecompound-not-throwing-error-with-invalid-syntax?utm_campaign=plugin&utm_content=tracker%2F14691067&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F14691067&utm_medium=issues&utm_source=github).