DanGould / rust-payjoin / 25482740670

85%

master: 85%

push

github

Enforce single-owner contract on InMemoryPersister

PR #1528 review feedback flagged that `Arc<RwLock<InnerStorage>>`
inside `Clone`-able `InMemoryPersister` invites concurrent-write
footgun: cloning the persister and sharing write access across
actors is a logic bug against the SessionPersister contract.

Drop `#[derive(Clone)]` and collapse internals to `Mutex<...>`.
Callers that genuinely need sharing must opt in via
`Arc<InMemoryPersister<V>>` and own the locking discipline.

Same change applies to `InMemoryAsyncPersister` (still cfg(test)).

32 of 32 new or added lines in 4 files covered. (100.0%)

11502 of 13507 relevant lines covered (85.16%)

400.3 hits per line