mongodb-js / devtools-shared / 16946613464 / 11 – download-url

78%

main: 78%

push

github

fix(devtools-connect): fix TLS error fallback mechanism MONGOSH-2488 (#566)

Restructure our proxy integration logic to:

- Avoid modifying the OIDC options – previously we were assigning to
  `clientOptions.oidc`, which meant that the `fetch` function created
  to read the system certificates was also used for the non-system-cert
  `connect()` call, breaking our fallback mechanism
- Add a test that verifies that the fallback mechanism works both for
  TLS driver connections and HTTPS OIDC calls.
- Deduplicate the logic for adding CA options to the `DevtoolsProxyOptions`
  instances.
- Share log info from the OIDC agent instance, if a new one has been
  created.
- Make sure that TLS errors passed through node-fetch are actually
  picked up by our "nested error" detection logic (they have a different
  `toStringTag` defined than regular `Error` instances).

275 of 350 branches covered (78.57%)

Branch coverage included in aggregate %.

271 of 320 relevant lines covered (84.69%)

560.86 hits per line