search

vapor / postgres-nio

🐘 Non-blocking, event-driven Swift client for PostgreSQL.
https://api.vapor.codes/postgresnio/documentation/postgresnio/
MIT License
317 stars 72 forks source link

Fixes crash in queries that timeout #351

Closed fabianfett closed 1 year ago

fabianfett commented 1 year ago

Fixes #347.

In an ideal case I add a unit test for this as well, but sadly this is rather involved.

cc @trasch.

codecov-commenter commented 1 year ago

Codecov Report

Merging #351 (590cd5d) into main (1516e0c) will decrease coverage by 0.01%. The diff coverage is 0.00%.

:exclamation: Current head 590cd5d differs from pull request most recent head 3b1db96. Consider uploading reports for the commit 3b1db96 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #351 +/- ## ========================================== - Coverage 41.20% 41.19% -0.01% ========================================== Files 117 117 Lines 9657 9658 +1 ========================================== Hits 3979 3979 - Misses 5678 5679 +1 ``` | [Impacted Files](https://codecov.io/gh/vapor/postgres-nio/pull/351?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vapor) | Coverage Δ | | |---|---|---| | [...urces/PostgresNIO/New/PostgresChannelHandler.swift](https://codecov.io/gh/vapor/postgres-nio/pull/351?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vapor#diff-U291cmNlcy9Qb3N0Z3Jlc05JTy9OZXcvUG9zdGdyZXNDaGFubmVsSGFuZGxlci5zd2lmdA==) | `61.93% <0.00%> (-0.14%)` | :arrow_down: |