Skip to content

Commit ab79844

Browse files
authored
Only retry errors if 429, 500 or 503 response codes during Partial Accept (#1005)
* Only retry errors if 429, 500 or 503 response codes during Partial Accept * Update test
1 parent e0d2ad4 commit ab79844

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

Library/Sender.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,8 @@ class Sender {
233233
let filteredEnvelopes: Contracts.EnvelopeTelemetry[] = [];
234234
if (breezeResponse.errors) {
235235
breezeResponse.errors.forEach(error => {
236-
if (this._isRetriable(error.statusCode)) {
236+
// Only retry errors if 429, 500 or 503 response codes
237+
if (error.statusCode == 429 || error.statusCode == 500 || error.statusCode == 503) {
237238
filteredEnvelopes.push(envelopes[error.index]);
238239
}
239240
});

Tests/Library/Sender.tests.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,10 +167,10 @@ describe("Library/Sender", () => {
167167
itemsReceived: 4,
168168
errors: [{
169169
index: 0,
170-
statusCode: 408,
170+
statusCode: 429,
171171
message: ""
172172
}, {
173-
index: 2,
173+
index: 1,
174174
statusCode: 123,
175175
message: ""
176176
}]

0 commit comments

Comments
 (0)