Code

Change span ID to use random bytes#654

Code

Merged

kjin merged 12 commits intogoogleapis:mastergoogleapis/cloud-trace-nodejs:masterfrom draffensperger:random-span-idsdraffensperger/cloud-trace-nodejs:random-span-idsCopy head branch name to clipboardJan 20, 2018

+65-21

Commits

Commits on Jan 18, 2018

• Change span ID to use random bytes

  draffensperger

  committed
  9c4a8f7

  Copy full SHA for 9c4a8f7

• Add disable ban comment to fix tslint

  draffensperger

  committed
  259cb12

  Copy full SHA for 259cb12

• Switch to use randomFillSync for performance

  draffensperger

  committed
  e221e5b

  Copy full SHA for e221e5b

• Use randomFillSync only if defined on crypto

  draffensperger

  committed
  1e6000d

  Copy full SHA for 1e6000d

• Fix tests and tidy up code

  draffensperger

  committed
  713f0ce

  Copy full SHA for 713f0ce

• Remove destructuring to fix test for node 4

  draffensperger

  committed
  430f37e

  Copy full SHA for 430f37e

• Remove const/let from test cases

  draffensperger

  committed
  998f696

  Copy full SHA for 998f696

Commits on Jan 19, 2018

• Use Math.random and strictEqual

  draffensperger

  committed
  6de97b7

  Copy full SHA for 6de97b7

• Fix SPAN_ID_MAX to be clearer

  draffensperger

  committed
  a422a57

  Copy full SHA for a422a57

• Go back to using crypto random bytes

  draffensperger

  committed
  6556d73

  Copy full SHA for 6556d73

• Fix formatting

  draffensperger

  committed
  f5df082

  Copy full SHA for f5df082

• Add back randomSpanId function I forgot

  draffensperger

  committed
  4f5c977

  Copy full SHA for 4f5c977