Max Bruckner
Max Bruckner
Reproducable on both `8cc129cff89523ea0f26ca23e3ca3de660041968` as well as `0.0.27`.
Just tested this with #2805 and this PR seems to fix that issue 🎉
Yup, pretty sure it's not flaky. If I test with `0.9.0-alpha.1`, I repeatably get between 50 and 80 warning logs about transaction already in progress or there not being any...
The only reason I even checked is this item in your list above: > * acquire() should now be completely cancel-safe.
It's perfectly possible that this PR doesn't fix all races that can lead to the issue in #2805, but it seems to have at least fixed the race that the...
This seems to be an issue with cancellation safety of transactions in sqlx. I found a self-contained way of reproducing the issue. `Cargo.toml`: ```toml [package] name = "sqlx_test" version =...
Note that in `axum` (and everything using `hyper` AFAIK), from what I understand every request is spawned as it's own task that is cancelled when the connection drops. I'm just...
I just checked if this can still be reproduced, and yes it can, both on `0.8.6` as well as latest `main` at the time of writing (`3abb1863244685eabc274eae665fe1c6a300a36e`)
Unfortunately not fixed in `0.9.0-alpha.1`.