Recovery: RecoveriesCollection.findRecoveryByShard should call recoveryStatus.tryIncRef before accessing fields

The function iterates over a snapshot of on going recoveries and thus may access RecoveryStatus objects that are already finished.

This issue was introduced with #8092 
can we invert this logic and do `if (recoveryStatus.tryIncRef()) { ` or use ` == false`