Improve error message for streaming on old host version (#226)

Author: ejizba

src/setup.ts

@@ -3,6 +3,7 @@
 
 import { SetupOptions } from '../types';
 import { AzFuncSystemError } from './errors';
+import { tryGetCoreApiLazy } from './utils/tryGetCoreApiLazy';
 import { workerSystemLog } from './utils/workerSystemLog';
 
 let options: SetupOptions = {};
@@ -16,6 +17,16 @@ export function setup(opts: SetupOptions): void {
     if (setupLocked) {
         throw new AzFuncSystemError("Setup options can't be changed after app startup has finished.");
     }
+
+    if (opts.enableHttpStream) {
+        // NOTE: coreApi.log was coincidentally added the same time as http streaming,
+        // so we can use that to validate the host version instead of messing with semver parsing
+        const coreApi = tryGetCoreApiLazy();
+        if (coreApi && !coreApi.log) {
+            throw new AzFuncSystemError(`HTTP streaming requires Azure Functions Host v4.28 or higher.`);
+        }
+    }
+
     options = opts;
     workerSystemLog('information', `Setup options: ${JSON.stringify(options)}`);
 }

src/utils/workerSystemLog.ts

@@ -8,7 +8,8 @@ import { tryGetCoreApiLazy } from './tryGetCoreApiLazy';
 
 export function workerSystemLog(level: types.LogLevel, ...args: unknown[]): void {
     const coreApi = tryGetCoreApiLazy();
-    if (coreApi) {
+    // NOTE: coreApi.log doesn't exist on older versions of the worker
+    if (coreApi && coreApi.log) {
         coreApi.log(level, 'system', format(...args));
     } else {
         fallbackLogHandler(level, ...args);