Latency of httpRequest not tracked when using library with redirectToStdout

[ktaekwon000]

1) Is this a client library issue or a product issue? This seems to be a client library issue.

2) Did someone already solve this? I have searched through issues in the listed repositories but I have been unable to find the issue I am facing.

3) Do you have a support contract? No.

If the support paths suggested above still do not result in a resolution, please provide the following details.

Environment details

• OS: Debian 12
• Node.js version: 18.16.1
• npm version: 9.5.1
• @google-cloud/logging-winston version: 5.3.0

Steps to reproduce

1. Log requests with the following code:

import * as gcpLogging from "@google-cloud/logging-winston";

const logger = createLogger({
  transports: [
    new gcpLogging.LoggingWinston({
      redirectToStdout: true,
    }),
  ],
  exitOnError: false,
});

const app = express();

app.use(await gcpLogging.express.makeMiddleware(logger));

2. In an environment with Cloud Logging Agent, make requests to this service. When browsing the logs in Log Explorer, it can be seen that httpRequest data such as status, size and user agent is tracked properly, but the latency (Duration object) is not tracked.

I believe this bug is due to a difference in schema where the cloud logging agent expects an output in the JSON format, but the library prints the output in the protobuf format, causing the logging agent to miss this value. (Related issue: https://github.com/googleapis/nodejs-logging-winston/issues/93)

[cindy-peng]

Thanks @ktaekwon000 for opening the issue. By design Logging client libraries call the gRPC Logging API, and the latency in HttpMessage is expected to be protobuf.

We cannot use latency as a single string (e.g. 3.5s) in our code since Duration is a protobuf structure expected by Cloud Logging service. I believe that the issue needs to be fixed in stackdriver-logging-agent.

The documentation with correct links will be fixed here: https://github.com/googleapis/nodejs-logging-winston/pull/828.