Revert "Adding the SDK Binding Support for Storage Blob (#341)" (#348)

* Revert "Adding the SDK Binding Support for Storage Blob (#341)"

This reverts commit 4845f8c.

* Updating Undici version

* Fix tests

---------

Co-authored-by: Swapnil Nagar <[email protected]>

Author: swapnil-nagar

.vscode/launch.json

@@ -14,15 +14,6 @@
             "request": "launch",
             "skipFiles": ["<node_internals>/**"],
             "type": "pwa-node"
-        },
-        {
-            "name": "Current TS Tests File",
-            "type": "node",
-            "request": "launch",
-            "program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
-            "args": ["-r", "ts-node/register", "${relativeFile}"],
-            "cwd": "${workspaceRoot}",
-            "protocol": "inspector"
         }
     ]
 }

package-lock.json

@@ -1,6 +1,6 @@
 {
     "name": "@azure/functions",
-    "version": "4.7.1-preview",
+    "version": "4.7.0",
     "description": "Microsoft Azure Functions NodeJS Framework",
     "keywords": [
         "azure",
@@ -41,10 +41,9 @@
         "watch": "webpack --watch --mode development"
     },
     "dependencies": {
-        "@azure/functions-extensions-base": "0.1.0-preview",
         "cookie": "^0.7.0",
         "long": "^4.0.0",
-        "undici": "^5.13.0"
+        "undici": "^5.29.0"
     },
     "devDependencies": {
         "@types/chai": "^4.2.22",
@@ -56,7 +55,6 @@
         "@types/mocha": "^9.1.1",
         "@types/node": "^18.0.0",
         "@types/semver": "^7.3.9",
-        "@types/sinon": "^17.0.4",
         "@typescript-eslint/eslint-plugin": "^5.12.1",
         "@typescript-eslint/parser": "^5.12.1",
         "chai": "^4.2.0",
@@ -67,8 +65,8 @@
         "eslint-plugin-header": "^3.1.1",
         "eslint-plugin-import": "^2.29.0",
         "eslint-plugin-prettier": "^4.0.0",
-        "eslint-plugin-simple-import-sort": "^10.0.0",
         "eslint-webpack-plugin": "^3.2.0",
+        "eslint-plugin-simple-import-sort": "^10.0.0",
         "fork-ts-checker-webpack-plugin": "^7.2.13",
         "fs-extra": "^10.0.1",
         "globby": "^11.0.0",
@@ -78,10 +76,9 @@
         "mocha-multi-reporters": "^1.5.1",
         "prettier": "^2.4.1",
         "semver": "^7.3.5",
-        "sinon": "^20.0.0",
         "ts-loader": "^9.3.1",
         "ts-node": "^3.3.0",
-        "typescript": "^4.9.5",
+        "typescript": "^4.5.5",
         "typescript4": "npm:typescript@~4.0.0",
         "webpack": "^5.74.0",
         "webpack-cli": "^4.10.0"

package.json

@@ -82,9 +82,11 @@ export class InvocationModel implements coreTypes.InvocationModel {
                 } else {
                     input = fromRpcTypedData(binding.data);
                 }
+
                 if (isTimerTrigger(bindingType)) {
                     input = toCamelCaseValue(input);
                 }
+
                 if (isTrigger(bindingType)) {
                     inputs.push(input);
                 } else {

src/InvocationModel.ts

@@ -1,6 +1,6 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the MIT License.
 
-export const version = '4.7.1-preview';
+export const version = '4.7.0';
 
 export const returnBindingKey = '$return';

src/constants.ts

@@ -2,7 +2,6 @@
 // Licensed under the MIT License.
 
 import { RpcTypedData } from '@azure/functions-core';
-import { ResourceFactoryResolver } from '@azure/functions-extensions-base';
 import { HttpRequest } from '../http/HttpRequest';
 import { isDefined } from '../utils/nonNull';
 
@@ -31,18 +30,8 @@ export function fromRpcTypedData(data: RpcTypedData | null | undefined): unknown
         return data.collectionDouble.double;
     } else if (data.collectionSint64 && isDefined(data.collectionSint64.sint64)) {
         return data.collectionSint64.sint64;
-    } else if (data.modelBindingData && isDefined(data.modelBindingData.content)) {
-        try {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call
-            const resourceFactoryResolver: ResourceFactoryResolver = ResourceFactoryResolver.getInstance();
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call
-            return resourceFactoryResolver.createClient(data.modelBindingData.source, data.modelBindingData);
-        } catch (exception) {
-            throw new Error(
-                'Unable to create client. Please register the extensions library with your function app. ' +
-                    `Error: ${exception instanceof Error ? exception.message : String(exception)}`
-            );
-        }
+    } else {
+        return undefined;
     }
 }
 

src/converters/fromRpcTypedData.ts

@@ -6,19 +6,17 @@ import * as coreTypes from '@azure/functions-core';
 import { returnBindingKey } from '../constants';
 import { AzFuncSystemError } from '../errors';
 import { isTrigger } from '../utils/isTrigger';
-import { workerSystemLog } from '../utils/workerSystemLog';
 import { toRpcDuration } from './toRpcDuration';
 
 export function toCoreFunctionMetadata(name: string, options: GenericFunctionOptions): coreTypes.FunctionMetadata {
     const bindings: Record<string, coreTypes.RpcBindingInfo> = {};
     const bindingNames: string[] = [];
-    const trigger = options.trigger;
 
+    const trigger = options.trigger;
     bindings[trigger.name] = {
         ...trigger,
         direction: 'in',
         type: isTrigger(trigger.type) ? trigger.type : trigger.type + 'Trigger',
-        properties: addSdkBindingsFlag(options.trigger?.sdkBinding, name, trigger.type, trigger.name, false),
     };
     bindingNames.push(trigger.name);
 
@@ -27,7 +25,6 @@ export function toCoreFunctionMetadata(name: string, options: GenericFunctionOpt
             bindings[input.name] = {
                 ...input,
                 direction: 'in',
-                properties: addSdkBindingsFlag(input?.sdkBinding, name, input.type, input.name, true),
             };
             bindingNames.push(input.name);
         }
@@ -77,45 +74,3 @@ export function toCoreFunctionMetadata(name: string, options: GenericFunctionOpt
 
     return { name, bindings, retryOptions };
 }
-
-/**
- * Adds the deferred binding flags to function bindings based on the binding configuration
- * @param sdkBindingType Boolean indicating if this is an SDK binding
- * @param functionName The name of the function for logging purposes
- * @param triggerType The type of the trigger or binding
- * @param bindingOrTriggerName The name of the trigger or binding
- * @param isBinding Boolean indicating if this is a binding (vs a trigger)
- * @returns Object with supportsDeferredBinding property set to 'true' or 'false'
- */
-export function addSdkBindingsFlag(
-    sdkBindingType?: boolean | unknown,
-    functionName?: string,
-    triggerType?: string,
-    bindingOrTriggerName?: string,
-    isBinding?: boolean
-): { [key: string]: string } {
-    // Ensure that trigger type is valid and supported
-    if (sdkBindingType !== undefined && sdkBindingType === true) {
-        const entityType = isBinding ? 'binding' : 'trigger';
-
-        // Create structured JSON log entry
-        const logData = {
-            operation: 'EnableDeferredBinding',
-            properties: {
-                functionName: functionName || 'unknown',
-                entityType: entityType,
-                triggerType: triggerType || 'unknown',
-                bindingOrTriggerName: bindingOrTriggerName || 'unknown',
-                supportsDeferredBinding: true,
-            },
-            message: `Enabled Deferred Binding of type '${triggerType || 'unknown'}' for function '${
-                functionName || 'unknown'
-            }'`,
-        };
-        // Log both the structured data
-        workerSystemLog('information', JSON.stringify(logData));
-        return { supportsDeferredBinding: 'true' };
-    }
-
-    return { supportsDeferredBinding: 'false' };
-}

src/converters/toCoreFunctionMetadata.ts

@@ -13,7 +13,7 @@ import {
     HttpTrigger,
     HttpTriggerOptions,
     MySqlTrigger,
-    MySqlTriggerOptions,
+    MySqlTriggerOptions,	
     ServiceBusQueueTrigger,
     ServiceBusQueueTriggerOptions,
     ServiceBusTopicTrigger,