@gradientedge/cdk-utils-azure 2.48.0 → 2.50.0

package/dist/src/construct/event-handler/main.d.ts

@@ -18,13 +18,10 @@ import { AzureEventHandlerProps, EventHandlerEventGridSubscription, EventHandler
  * | `true`  | `true`  | Look up both. Cross-stack pattern where the producer/owner of the queue is a different stack (e.g. `WebhookEventHandler` consuming a queue created by `WebhookGateway`). |
  * | `false` | `true`  | **Invalid** — construct-time error. You cannot resolve an existing queue under a namespace the construct is creating. |
  *
- * The top-level `serviceBus.useExisting` flag is retained as a deprecated alias that sets both
- * per-resource flags to the same value, so existing callers continue to work unchanged.
- *
  * ## Authorization and the `EVENT_INGEST_SERVICE_BUS` connection string
  *
  * When the construct owns the queue (`queue.useExisting=false`), it provisions a per-queue
- * authorization rule named `${id}-listen-send` with `Listen + Send` rights, and the function app's
+ * authorization rule named `listen-send` (scoped to the queue) with `Listen + Send` rights, and the function app's
  * `EVENT_INGEST_SERVICE_BUS` connection string is sourced from that rule. This avoids granting the
  * function app access to sibling queues when the namespace is shared.
  *
@@ -99,8 +96,6 @@ export declare class AzureEventHandler extends AzureFunctionApp {
     /**
      * @summary Resolve effective `useExisting` flags for the Service Bus namespace and queue.
      *
-     * Per-resource flags (`namespace.useExisting`, `queue.useExisting`) take precedence over the
-     * deprecated top-level `serviceBus.useExisting`, which is treated as an alias that sets both.
      * Throws if the invalid combination (namespace.useExisting=false + queue.useExisting=true) is
      * requested — a queue cannot be looked up under a namespace the construct is about to create.
      */

package/dist/src/construct/event-handler/main.js

@@ -19,13 +19,10 @@ import { AzureFunctionApp } from '../function-app/index.js';
  * | `true`  | `true`  | Look up both. Cross-stack pattern where the producer/owner of the queue is a different stack (e.g. `WebhookEventHandler` consuming a queue created by `WebhookGateway`). |
  * | `false` | `true`  | **Invalid** — construct-time error. You cannot resolve an existing queue under a namespace the construct is creating. |
  *
- * The top-level `serviceBus.useExisting` flag is retained as a deprecated alias that sets both
- * per-resource flags to the same value, so existing callers continue to work unchanged.
- *
  * ## Authorization and the `EVENT_INGEST_SERVICE_BUS` connection string
  *
  * When the construct owns the queue (`queue.useExisting=false`), it provisions a per-queue
- * authorization rule named `${id}-listen-send` with `Listen + Send` rights, and the function app's
+ * authorization rule named `listen-send` (scoped to the queue) with `Listen + Send` rights, and the function app's
  * `EVENT_INGEST_SERVICE_BUS` connection string is sourced from that rule. This avoids granting the
  * function app access to sibling queues when the namespace is shared.
  *
@@ -137,20 +134,12 @@ export class AzureEventHandler extends AzureFunctionApp {
     /**
      * @summary Resolve effective `useExisting` flags for the Service Bus namespace and queue.
      *
-     * Per-resource flags (`namespace.useExisting`, `queue.useExisting`) take precedence over the
-     * deprecated top-level `serviceBus.useExisting`, which is treated as an alias that sets both.
      * Throws if the invalid combination (namespace.useExisting=false + queue.useExisting=true) is
      * requested — a queue cannot be looked up under a namespace the construct is about to create.
      */
     resolveServiceBusUseExisting() {
-        // TODO: remove `deprecatedServiceBusUseExisting` and the `?? deprecatedServiceBusUseExisting`
-        //       fallbacks once all callers have migrated to the per-resource flags
-        //       (see EventHandlerServiceBusProps.useExisting in types.ts).
-        const deprecatedServiceBusUseExisting = this.props.serviceBus?.useExisting;
-        const namespaceUseExisting = this.props.serviceBus?.namespace?.useExisting;
-        const queueUseExisting = this.props.serviceBus?.queue?.useExisting;
-        const namespace = namespaceUseExisting ?? deprecatedServiceBusUseExisting ?? false;
-        const queue = queueUseExisting ?? deprecatedServiceBusUseExisting ?? false;
+        const namespace = this.props.serviceBus?.namespace?.useExisting ?? false;
+        const queue = this.props.serviceBus?.queue?.useExisting ?? false;
         if (!namespace && queue) {
             throw new Error(`[${this.id}] invalid serviceBus configuration: queue.useExisting=true requires namespace.useExisting=true ` +
                 `(cannot resolve an existing queue under a namespace the construct is creating).`);
@@ -231,8 +220,11 @@ export class AzureEventHandler extends AzureFunctionApp {
         const namespaceResourceGroupName = useExistingFlags.namespace
             ? (this.props.serviceBus?.namespace?.resourceGroupName ?? this.resourceGroup.name)
             : this.resourceGroup.name;
+        // Azure caps `authorizationRuleName` at 50 chars. The rule's scope is the queue itself
+        // (`…/namespaces/<ns>/queues/<queue>/authorizationRules/<rule>`), so a literal name is
+        // unambiguous and avoids hitting the cap on long stack ids.
         this.serviceBus.queueAuthorizationRule = this.serviceBusManager.createServiceBusQueueAuthorizationRule(this.id, this, {
-            authorizationRuleName: `${this.id}-listen-send`,
+            authorizationRuleName: 'listen-send',
             namespaceName: this.serviceBus.namespace.name,
             queueName: this.serviceBus.queue.name,
             resourceGroupName: namespaceResourceGroupName,

package/dist/src/construct/event-handler/types.d.ts

@@ -61,14 +61,6 @@ export interface EventHandlerServiceBusProps {
     namespace?: EventHandlerServiceBusNamespaceProps;
     /** Service Bus queue properties (extends {@link ServiceBusQueueProps} with `useExisting`) */
     queue?: EventHandlerServiceBusQueueProps;
-    /**
-     * Convenience alias that sets both `namespace.useExisting` and `queue.useExisting` to the same value.
-     * @deprecated Prefer `namespace.useExisting` and `queue.useExisting` individually. Retained as an alias
-     * so existing callers (e.g. WebhookEventHandler) continue to compile unchanged.
-     * TODO: remove once all callers have migrated to the per-resource flags. Also remove the
-     * resolution fallback in `resolveServiceBusUseExisting()` in main.ts.
-     */
-    useExisting?: boolean;
 }
 /**
  * Provisioned Service Bus resources for the event handler

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gradientedge/cdk-utils-azure",
-  "version": "2.48.0",
+  "version": "2.50.0",
   "description": "Azure Pulumi utilities for @gradientedge/cdk-utils",
   "type": "module",
   "main": "dist/src/index.js",