json-everything: NullReference Exception after updating to 5.1.3, 5.2.0, 5.2.1

Nuget Package

JsonSchema.Net

Package Version

5.1.3, 5.2.0, 5.2.1

Operating System

Linux

.Net Target (if relevant, please specify the version in the description)

.Net (5 or after)

A clear and concise description of the bug

We upgrated from 4.1.8 to 5.1.2 and since then we get the following error.

System.NullReferenceException: Object reference not set to an instance of an object.
   at Json.Schema.SchemaEvaluation.FindEvaluation(Guid id)
   at Json.Schema.KeywordConstraint.BuildEvaluation(SchemaEvaluation schemaEvaluation, JsonPointer instanceLocation, JsonPointer evaluationPath)
   at Json.Schema.SchemaConstraint.BuildEvaluation(JsonNode localInstance, JsonPointer instanceLocation, JsonPointer evaluationPath, EvaluationOptions options)
   at Json.Schema.JsonSchema.Evaluate(JsonNode root, EvaluationOptions options)

Formatted by https://st.elmah.io

Please add test code or steps to reproduce the behavior

We were not able to reproduce it yet. We work on it.

What did you expect?

That it works.

Is there any other information you’d like to share regarding this bug?

No response

Code of Conduct

I agree to follow this project’s Code of Conduct

About this issue

Original URL
State: closed
Created a year ago
Comments: 26 (13 by maintainers)

Most upvoted comments

@gregsdennis I tried to setup a separate project to repro so that I could post it, but I haven’t gotten it to repro. It still repros in the other project. I’m trying to narrow down the difference. I’ll post a project once I get it to repro there.

edaena on Sep 7, 2023

Hi @gregsdennis Sure, just wrote it because of the comment of @mdekrey. Saw some concurrency fixes and thought it’s worth a try. Still on our backlog to create a standalone repro.

msallin on Sep 3, 2023

They aren’t. Give us some time and we will try every schema in a test to see if we can reproduce this. If we find that one schema is breaking the code, we will clean all sensitive info and post it here.

msallin on Aug 9, 2023