Skip to content

Commit

Permalink
Throw exception on null definition
Browse files Browse the repository at this point in the history
  • Loading branch information
LTA-Thinking committed Jan 4, 2024
1 parent 48cd72a commit c260875
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Microsoft.Health.TaskManagement/JobInfoExtensions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ public static T DeserializeDefinition<T>(this JobInfo jobInfo)
{
EnsureArg.IsNotNull(jobInfo, nameof(jobInfo));

if (jobInfo.Definition == null)
if (jobInfo.Definition == null || jobInfo.Definition.Equals("null", StringComparison.OrdinalIgnoreCase))
{
return default;
throw new ArgumentNullException(nameof(jobInfo));
}

return JsonConvert.DeserializeObject<T>(jobInfo.Definition);
Expand Down

0 comments on commit c260875

Please sign in to comment.