Possible replication of bug from nodejs-logging-bunyan

[yuku]

It appears that a bug within the nodejs-logging-winston repository mirrors a previously resolved issue in the nodejs-logging-bunyan repository, specifically issue googleapis/nodejs-logging-bunyan#626 - Duplicate HTTP request entries when running in Cloud Run.

The root of this problem seems to lie within this block of code in nodejs-logging-winston:

nodejs-logging-winston/src/middleware/express.ts

Line 83 in 9b8657a

if (env !== GCPEnv.APP_ENGINE && env !== GCPEnv.CLOUD_FUNCTIONS) {

This block is identical to the code previously found in nodejs-logging-bunyan before the bug was fixed with pull request googleapis/nodejs-logging-bunyan#658 (link):

  if (env !== GCPEnv.APP_ENGINE && env !== GCPEnv.CLOUD_FUNCTIONS) {

Following the implementation of the pull request in nodejs-logging-bunyan, the code was amended to (link):

  if (
    env !== GCPEnv.APP_ENGINE &&
    env !== GCPEnv.CLOUD_FUNCTIONS &&
    (env !== GCPEnv.CLOUD_RUN || !options.skipParentEntryForCloudRun)
  ) {

It seems plausible that applying a similar fix in nodejs-logging-winston could resolve this issue.