16828478718
99%

Ran 08 Aug 2025 10:53AM UTC

Jobs 1

Files 293

Run time 263min

Badge

Embed ▾

Committed 08 Aug 2025 10:45AM UTC coverage: 97.889%. Remained the same

Build # 16828478718

Build Type

push

github

Committed by

web-flow

Commit Message

Fix IdbStorage "Arc::try_unwrap failed" on multiple connections (#1718)

The `on_upgrade_needed` callback for IndexedDB open requests was holding a reference to an `Arc`, which was not being released when the `IdbStorage` instance was dropped. This caused `Arc::try_unwrap` to fail when a new `IdbStorage` instance was created with the same namespace.

This commit fixes the issue by replacing the `Arc::try_unwrap` logic with a safer approach that locks the mutex and takes the error value. This avoids the panic while still correctly handling errors from the callback.

A test case has been added to reproduce the issue and verify the fix.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

Coverage Stats

39748 of 40605 relevant lines covered (97.89%)

60544.75 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	16828478718.1	08 Aug 2025 10:53AM UTC	293	97.89	GitHub Action Run

gluesql / gluesql / 16828478718
99%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 16828478718

gluesql / gluesql / 16828478718 99%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 16828478718

gluesql / gluesql / 16828478718
99%

README BADGES
x