Skip to content

Commit 8d5911d

Browse files
committed
fix retry in event triggers
1 parent 3e7a4b7 commit 8d5911d

File tree

9 files changed

+9
-7
lines changed

9 files changed

+9
-7
lines changed

src/v2/providers/alerts/alerts.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@ export function getEndpointAnnotation(
249249
eventFilters: {
250250
alerttype: alertType,
251251
},
252-
retry: !!opts.retry,
252+
retry: opts.retry || false,
253253
},
254254
};
255255
if (appId) {

src/v2/providers/database.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -446,7 +446,7 @@ export function makeEndpoint(
446446
eventType,
447447
eventFilters,
448448
eventFilterPathPatterns,
449-
retry: false,
449+
retry: opts.retry || false,
450450
},
451451
};
452452
}

src/v2/providers/eventarc.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ export function onCustomEventPublished<T = any>(
216216
eventTrigger: {
217217
eventType: opts.eventType,
218218
eventFilters: {},
219-
retry: false,
219+
retry: opts.retry || false,
220220
channel,
221221
},
222222
};

src/v2/providers/firestore.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -426,7 +426,7 @@ export function makeEndpoint(
426426
eventType,
427427
eventFilters,
428428
eventFilterPathPatterns,
429-
retry: !!opts.retry,
429+
retry: opts.retry || false,
430430
},
431431
};
432432
}

src/v2/providers/pubsub.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -324,6 +324,7 @@ export function onMessagePublished<T = any>(
324324
eventTrigger: {
325325
eventType: "google.cloud.pubsub.topic.v1.messagePublished",
326326
resource: `projects/${process.env.GCLOUD_PROJECT}/topics/${topic}`,
327+
retry: opts.retry || false,
327328
},
328329
};
329330
},

src/v2/providers/remoteConfig.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ export function onConfigUpdated(
145145
eventTrigger: {
146146
eventType,
147147
eventFilters: {},
148-
retry: !!optsOrHandler.retry,
148+
retry: optsOrHandler.retry || false,
149149
},
150150
};
151151
func.__endpoint = ep;

src/v2/providers/storage.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -603,7 +603,7 @@ export function onOperation(
603603
eventTrigger: {
604604
eventType,
605605
eventFilters: { bucket },
606-
retry: false,
606+
retry: opts.retry || false,
607607
},
608608
};
609609
copyIfPresent(endpoint.eventTrigger, opts, "retry", "retry");

src/v2/providers/tasks.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -281,6 +281,7 @@ export function onTaskDispatched<Args = any>(
281281
convertInvoker
282282
);
283283
convertIfPresent(func.__endpoint.taskQueueTrigger, opts, "invoker", "invoker", convertInvoker);
284+
copyIfPresent(func.__endpoint.taskQueueTrigger, opts, "retry", "retry")
284285

285286
func.__requiredAPIs = [
286287
{

src/v2/providers/testLab.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ export function onTestMatrixCompleted(
206206
eventTrigger: {
207207
eventType,
208208
eventFilters: {},
209-
retry: !!optsOrHandler.retry,
209+
retry: optsOrHandler.retry || false,
210210
},
211211
};
212212
func.__endpoint = ep;

0 commit comments

Comments
 (0)