KeyManagementServiceClient is leaked on each call to sign or getPublicKey

[nedgar]

In src/util/gcp-kms-utils.ts it instantiates a new KeyManagementServiceClient each time sign or getPublicKey is called.
See:

ethers-gcp-kms-signer/src/util/gcp-kms-utils.ts

Line 25 in 45a5186

const kms = new KeyManagementServiceClient();

ethers-gcp-kms-signer/src/util/gcp-kms-utils.ts

Line 43 in 45a5186

const kms = new KeyManagementServiceClient();

The client maintains a gRPC channel, plus one in its referenced IAM client. close() should be called on the client when it's no longer needed.

The logical lifetime of the client is that of the GcpKmsSigner. I suggest changing it to instantiate the KeyManagementServiceClient, and pass that to the util methods.

[nedgar]

Alternatively, since GcpKmsSigner doesn't take the Google credentials, the KeyManagementServiceClient could be made a singleton field in the utils for now. At least that would reduce leakage to just that instance and its channels.