fix: recover from broken connection
Fixes https://github.com/knex/knex/issues/5732
@OlivierCavadenti or @kibertoad is there anything needed to merge this ? Not having this fix can lead to a backend not responding at all, because some events (ex: connection kill) would lead to broken connection clogging the pool.
@kibertoad Kindly ask again to merge this. As said before, not having this fix can lead to a backend not responding at all, because some events (ex: alter system kill session) would prevent to dispose an unusable connection when validating this connection .
It would be great to add NJS-521 and NJS-500 as part of PR. We hit this same issue when connecting to OCI. As a workaround, leveraged pnpm patch <pkg name>@<version> to apply patch and seems to work very well.
Happy to offer my services to help push this along.
Thank you maintainers!
@rluvaton Could you please review?
This looks good, thank you!
Can we please have a patch tag/release and push a new npm package for this change to be released? Thank you!