handle nullref for empty host.json. Closes #101

ahmelsayed · ahmelsayed · commit 8726bf3d7c87 · 2017-04-04T15:56:07.000-07:00
diff --git a/src/Azure.Functions.Cli/Actions/HostActions/StartHostAction.cs b/src/Azure.Functions.Cli/Actions/HostActions/StartHostAction.cs
@@ -150,7 +150,10 @@ private void CheckHostJsonId()
         {
             if (FileSystemHelpers.FileExists(ScriptConstants.HostMetadataFileName))
             {
-                var hostConfig = JsonConvert.DeserializeObject<JObject>(FileSystemHelpers.ReadAllTextFromFile(ScriptConstants.HostMetadataFileName));
+                var hostJsonContent = FileSystemHelpers.ReadAllTextFromFile(ScriptConstants.HostMetadataFileName);
+                var hostConfig = string.IsNullOrEmpty(hostJsonContent)
+                    ? new JObject()
+                    : JsonConvert.DeserializeObject<JObject>(FileSystemHelpers.ReadAllTextFromFile(ScriptConstants.HostMetadataFileName));
                 if (hostConfig["id"] == null)
                 {
                     ColoredConsole.Out

Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,10 @@ private void CheckHostJsonId()`
`150`	`150`	`{`
`151`	`151`	`if (FileSystemHelpers.FileExists(ScriptConstants.HostMetadataFileName))`
`152`	`152`	`{`
`153`		`- var hostConfig = JsonConvert.DeserializeObject<JObject>(FileSystemHelpers.ReadAllTextFromFile(ScriptConstants.HostMetadataFileName));`
	`153`	`+ var hostJsonContent = FileSystemHelpers.ReadAllTextFromFile(ScriptConstants.HostMetadataFileName);`
	`154`	`+ var hostConfig = string.IsNullOrEmpty(hostJsonContent)`
	`155`	`+ ? new JObject()`
	`156`	`+ : JsonConvert.DeserializeObject<JObject>(FileSystemHelpers.ReadAllTextFromFile(ScriptConstants.HostMetadataFileName));`
`154`	`157`	`if (hostConfig["id"] == null)`
`155`	`158`	`{`
`156`	`159`	`ColoredConsole.Out`