Skip to content

fix(http_pool): add retry logging - #6

Open
capy-ai[bot] wants to merge 1 commit into
mainfrom
capy/http-pool-retry-logging
Open

fix(http_pool): add retry logging#6
capy-ai[bot] wants to merge 1 commit into
mainfrom
capy/http-pool-retry-logging

Conversation

@capy-ai

@capy-ai capy-ai Bot commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

This PR adds logging to the HTTP connection pool retry logic in src/http_pool.nim, which previously swallowed BadClientError and ProtocolError exceptions silently. The retry behavior remains unchanged — only visibility is improved.

Changes

  • src/http_pool.nim: Log retry attempts with exception name and message using [http_pool] prefix
  • src/http_pool.nim: Wrap retry in nested try/except to log and re-raise if retry also fails

Open in Capy SCO-7 · 5.4

@capy-ai capy-ai Bot added the capy Generated by capy.ai label Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

capy Generated by capy.ai

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant