Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: tighten HTTP status regex to avoid matching longer codes, fix comment to mention 408 exclusion

Browse Source
This commit is contained in:
Benedikt Schackenberg 2026-03-19 20:43:41 +00:00
parent 5a0369197e
commit 996a1d5e63
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/infra/outbound/delivery-queue.ts
View File

@ -442,13 +442,13 @@ const PERMANENT_ERROR_PATTERNS: readonly RegExp[] = [
* Matches HTTP status codes embedded in error strings, e.g.
* "status 400", "HTTP 403", "Status code: 413", "status code 400".
*/
const HTTP_STATUS_PATTERN = /(?:status(?:\s+code)?:?\s*|HTTP\/?\s*)(\d{3})/i;
const HTTP_STATUS_PATTERN = /(?:status(?:\s+code)?:?\s*|HTTP\/?\s*)(\d{3})(?!\d)/i;
export function isPermanentDeliveryError(error: string): boolean {
if (PERMANENT_ERROR_PATTERNS.some((re) => re.test(error))) {
return true;
}
// Detect HTTP 4xx client errors (permanent), except 429 (rate limit).
// Detect HTTP 4xx client errors (permanent), except 408 (timeout) and 429 (rate limit).
const statusMatch = HTTP_STATUS_PATTERN.exec(error);
if (statusMatch) {
const status = parseInt(statusMatch[1], 10);