Support secretless extensions in V3 (#2713)

* merge conflict resolved

* more merge conflicts

* fix tests

Author: madelinegordon

src/Azure.Functions.Cli/Actions/HostActions/StartHostAction.cs

@@ -415,7 +415,7 @@ private bool IsPreCompiledFunctionApp()
             return isPrecompiled;
         }
 
-        internal static async Task CheckNonOptionalSettings(IEnumerable<KeyValuePair<string, string>> secrets, string scriptPath, bool userSecretsEnabled)
+        internal static async Task CheckNonOptionalSettings(IEnumerable<KeyValuePair<string, string>> secrets, string scriptPath, bool skipAzureWebJobsStorageCheck = false)
         {
             string storageConnectionKey = "AzureWebJobsStorage";
             try
@@ -441,15 +441,12 @@ internal static async Task CheckNonOptionalSettings(IEnumerable<KeyValuePair<str
                     .Where(b => b.IndexOf("Trigger", StringComparison.OrdinalIgnoreCase) != -1)
                     .All(t => Constants.TriggersWithoutStorage.Any(tws => tws.Equals(t, StringComparison.OrdinalIgnoreCase)));
 
-                if (string.IsNullOrWhiteSpace(azureWebJobsStorage) &&
-                    !StorageConnectionExists(secrets, storageConnectionKey) &&
-                    !allNonStorageTriggers)
+                if (!skipAzureWebJobsStorageCheck && string.IsNullOrWhiteSpace(azureWebJobsStorage) && 
+                    !StorageConnectionExists(secrets, storageConnectionKey) && !allNonStorageTriggers) 
                 {
-                    string errorMessage = userSecretsEnabled ? Constants.Errors.WebJobsStorageNotFoundWithUserSecrets : Constants.Errors.WebJobsStorageNotFound;
-                    throw new CliException(string.Format(errorMessage,
-                                                         SecretsManager.AppSettingsFileName,
-                                                         string.Join(", ", Constants.TriggersWithoutStorage),
-                                                         SecretsManager.AppSettingsFileName));
+                    throw new CliException($"Missing value for AzureWebJobsStorage in {SecretsManager.AppSettingsFileName}. " +
+                        $"This is required for all triggers other than {string.Join(", ", Constants.TriggersWithoutStorage)}. "
+                        + $"You can run 'func azure functionapp fetch-app-settings <functionAppName>', specify a connection string in {SecretsManager.AppSettingsFileName}, or use managed identity to authenticate.");
                 }
 
                 foreach ((var filePath, var functionJson) in functionsJsons)
@@ -467,14 +464,9 @@ internal static async Task CheckNonOptionalSettings(IEnumerable<KeyValuePair<str
                                 }
                                 else if (!secrets.Any(v => v.Key.Equals(appSettingName, StringComparison.OrdinalIgnoreCase)))
                                 {
-                                    string warningMessage = userSecretsEnabled ? Constants.Errors.AppSettingNotFoundWithUserSecrets : Constants.Errors.AppSettingNotFound;
-                                    ColoredConsole.WriteLine(WarningColor(string.Format(warningMessage,
-                                                                                        appSettingName,
-                                                                                        SecretsManager.AppSettingsFileName,
-                                                                                        token.Key,
-                                                                                        binding["type"]?.ToString(),
-                                                                                        filePath,
-                                                                                        SecretsManager.AppSettingsFileName)));
+                                    ColoredConsole
+                                        .WriteLine(WarningColor($"Warning: Cannot find value named '{appSettingName}' in {SecretsManager.AppSettingsFileName} that matches '{token.Key}' property set on '{binding["type"]?.ToString()}' in '{filePath}'. " +
+                                            $"You can run 'func azure functionapp fetch-app-settings <functionAppName>' or specify a connection string in {SecretsManager.AppSettingsFileName}."));
                                 }
                             }
                         }

test/Azure.Functions.Cli.Tests/ActionsTests/StartHostActionTests.cs

@@ -69,7 +69,7 @@ public async Task CheckNonOptionalSettingsDoesntThrowMissingStorageUsingManagedI
             Exception exception = null;
             try
             {
-                await StartHostAction.CheckNonOptionalSettings(secrets, "x:\\", false);
+                await StartHostAction.CheckNonOptionalSettings(secrets, "x:\\");
             }
             catch (Exception e)
             {
@@ -81,24 +81,31 @@ public async Task CheckNonOptionalSettingsDoesntThrowMissingStorageUsingManagedI
         [Fact]
         public async Task CheckNonOptionalSettingsDoesntThrowOnMissingAzureWebJobsStorage()
         {
+            Skip.IfNot(RuntimeInformation.IsOSPlatform(OSPlatform.Windows),
+                reason: "Environment.CurrentDirectory throws in linux in test cases for some reason. Revisit this once we figure out why it's failing");
             var fileSystem = GetFakeFileSystem(new[]
-            {
-                ("x:\\folder1", "{'bindings': [{'type': 'httpTrigger'}]}"),
+                {
+                ("x:\\folder1", "{'bindings': [{'type': 'blobTrigger'}]}"),
                 ("x:\\folder2", "{'bindings': [{'type': 'httpTrigger'}]}")
             });
 
+            var secrets = new Dictionary<string, string>()
+            {
+                { "AzureWebJobsStorage:blobServiceUri", "myuri" },
+                { "AzureWebJobsStorage__queueServiceUri", "queueuri" }
+            };
+
             FileSystemHelpers.Instance = fileSystem;
 
             Exception exception = null;
             try
             {
-                await StartHostAction.CheckNonOptionalSettings(new Dictionary<string, string>(), "x:\\", false);
+                await StartHostAction.CheckNonOptionalSettings(secrets, "x:\\", false);
             }
             catch (Exception e)
             {
                 exception = e;
             }
-
             exception.Should().BeNull();
         }