fix: Fix trace_sampled type to be consistent with API documentation (#830)

* fix: Fix timestamp drift due to prototype comparison

* fix: Fix traceSampled to be boolean as documented

* Revert timestamp change

---------

Co-authored-by: cindy-peng <cindypeng@google.com>
Co-authored-by: meredithslota <meredithslota@google.com>

Author: 3 people

packages/nodejs-logging-winston/src/common.ts

@@ -277,7 +277,7 @@ export class LoggingCommon {
     }
 
     if (LOGGING_SAMPLED_KEY in metadata) {
-      entryMetadata.traceSampled = metadata[LOGGING_SAMPLED_KEY] === '1';
+      entryMetadata.traceSampled = metadata[LOGGING_SAMPLED_KEY] === true;
     }
 
     // we have tests that assert that metadata is always passed.

packages/nodejs-logging-winston/test/common.ts

@@ -395,7 +395,7 @@ describe('logging-common', () => {
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       (metadataWithTrace as any)[loggingSpanKey] = 'span1';
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      (metadataWithTrace as any)[loggingSampledKey] = '1';
+      (metadataWithTrace as any)[loggingSampledKey] = true;
 
       loggingCommon.cloudLog.entry = (entryMetadata: {}, data: {}) => {
         assert.deepStrictEqual(entryMetadata, {

packages/nodejs-logging-winston/test/middleware/make-child-logger.ts

@@ -120,7 +120,7 @@ describe('makeChildLogger', () => {
     child.debug('hello world', {
       [LOGGING_TRACE_KEY]: 'to-be-clobbered',
       [LOGGING_SPAN_KEY]: 'to-be-clobbered',
-      [LOGGING_SAMPLED_KEY]: '0',
+      [LOGGING_SAMPLED_KEY]: false,
     });
     assert.notStrictEqual(trace, FAKE_TRACE);
     assert.notStrictEqual(span, FAKE_SPAN);