This can be retried

[dgageot]

No description provided.

[docker-agent]

Review Summary

Assessment: 🟢 APPROVE

Summary

This PR adds "internal_error" as a retryable pattern to handle HTTP/2 INTERNAL_ERROR stream-level errors. The implementation is correct and follows existing patterns in the codebase.

Analysis:

• ✅ The pattern correctly uses case-insensitive matching (via strings.ToLower())
• ✅ Multi-layer defense prevents false positives: HTTP status codes are extracted first, then nonRetryablePatterns are checked
• ✅ The pattern follows precedent of other broad patterns like "timeout", "connection reset", "overloaded"
• ✅ Test case validates the positive case with a realistic HTTP/2 error format
• ✅ Code comment documents the intent clearly

No issues found. The changed code works correctly and is ready to merge.