• grrgyle@slrpnk.net
    link
    fedilink
    arrow-up
    8
    ·
    2 months ago

    Just skimmed but seems like a decent idea. Not that I’ve knowingly run into issues parsing JSON too much

    • towerful
      link
      fedilink
      arrow-up
      8
      arrow-down
      1
      ·
      2 months ago

      It’s from 2015, so its probably what you are doing anyway

      • lysdexicOP
        link
        fedilink
        English
        arrow-up
        3
        arrow-down
        1
        ·
        edit-2
        2 months ago

        It’s from 2015, so its probably what you are doing anyway

        No, you are probably not using this at all. The problem with JSON is that this details are all handled in an implementation-defined way, and most implementation just fail/round silently.

        Just give it a try and send down the wire a JSON with, say, a huge integer, and see if that triggers a parsing error. For starters, in .NET both Newtonsoft and System.Text.Json set a limit of 64 bits.

        https://learn.microsoft.com/en-us/dotnet/api/system.text.json.jsonserializeroptions.maxdepth