Jarom Loveridge

Warnings emitted on Node 14

7

When appmetrics is required on node 14 multiple messages are emitted which state: ``` (node:74412) Warning: Accessing non-existent property '__ddProbeAttached__' of module exports inside circular dependency ``` The following simple...

I noticed the `Dockerfile` for the alpine images have the following in them: ``` # Get Dependencies COPY ./nsolid-bundle-${NSOLID_VERSION}/nsolid-*${NODEJS_LTS}-alpine-x64.tar.gz . # later it does the following in a RUN command...

Allow passing MongoDB Native options when calling connect

1

Sometimes it is useful to pass additional options to the driver when connecting. Specifically we needed to be able to pass options that affect the SSL connection since require certificate...

Support expanded support for converting to boolean values.

1

Be a bit more permissive on what values will result in a `true` from the boolean conversion. Additionally this fixes an issue where an exception is thrown if the value...

Make `convert` a utility method rather than private class member

2

While I don't particularly like prefixing with `__` as a method of hiding private methods I think it best to avoid using any kind of name that might reasonably be...

The `reflect-metadata` package should be a peer dependency.

1

I have encountered issues when multiple dependencies of a project depend on the `reflect-metadata` package. When this occurs I have seen `tsc` fail to transpile the project source. This has,...

Add support for obtaining sync tokens

5

Adds `OAuth` class and `me` method to the exports. The intent is to allow someone to implement their Instant Login and Secure Sync with the same library. Also adds support...

Add test to verify end is called at the appropriate time.

2

Add a testcase which demonstrates that 'end' is being called while a stream is paused. This can be problematic if you are doing async processing of data. In my case...