dd-trace 2.0.0-appsec-alpha.1 → 2.0.0-appsec-beta.1

package/LICENSE-3rdparty.csv

@@ -20,6 +20,7 @@ require,multer,MIT,Copyright 2014 Hage Yaapa
 require,opentracing,MIT,Copyright 2016 Resonance Labs Inc
 require,path-to-regexp,MIT,Copyright 2014 Blake Embrey
 require,performance-now,MIT,Copyright 2013 Braveg1rl
+require,retry,MIT,Copyright 2011 Tim Koschützki Felix Geisendörfer
 require,semver,ISC,Copyright Isaac Z. Schlueter and Contributors
 require,source-map,BSD-3-Clause,Copyright 2009-2011 Mozilla Foundation and contributors
 require,source-map-resolve,MIT,Copyright 2014-2020 Simon Lydell 2019 Jinxiang
@@ -53,7 +54,6 @@ dev,msgpack-lite,MIT,Copyright 2015 Yusuke Kawasaki
 dev,nock,MIT,Copyright 2017 Pedro Teixeira and other contributors
 dev,nyc,ISC,Copyright 2015 Contributors
 dev,proxyquire,MIT,Copyright 2013 Thorsten Lorenz
-dev,retry,MIT,Copyright 2011 Tim Koschützki Felix Geisendörfer
 dev,rimraf,ISC,Copyright Isaac Z. Schlueter and Contributors
 dev,sinon,BSD-3-Clause,Copyright 2010-2017 Christian Johansen
 dev,sinon-chai,WTFPL and BSD-2-Clause,Copyright 2004 Sam Hocevar 2012–2017 Domenic Denicola

package/index.d.ts

@@ -285,12 +285,6 @@ export declare interface TracerOptions {
    */
   runtimeMetrics?: boolean
 
-  /**
-   * Whether to track the scope of async functions. This is needed for async/await to work with non-native promises (thenables). Only disable this if you are sure only native promises are used with async/await, or if you are using Node >=14.5 since the issue has been fixed in that version.
-   * @default true
-   */
-  trackAsyncScope?: boolean
-
   /**
    * Custom function for DNS lookups when sending requests to the agent.
    * @default dns.lookup()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dd-trace",
-  "version": "2.0.0-appsec-alpha.1",
+  "version": "2.0.0-appsec-beta.1",
   "description": "Datadog APM tracing client for JavaScript",
   "main": "index.js",
   "typings": "index.d.ts",
@@ -16,8 +16,10 @@
     "services": "node ./scripts/install_plugin_modules && node packages/dd-trace/test/setup/services",
     "tdd": "node scripts/tdd.js",
     "test": "SERVICES=* yarn services && mocha --exit --expose-gc 'packages/dd-trace/test/setup/node.js' 'packages/*/test/**/*.spec.js'",
-    "test:core": "mocha --exit --expose-gc --file packages/dd-trace/test/setup/core.js \"packages/dd-trace/test/**/*.spec.js\"",
-    "test:core:ci": "nyc --include \"packages/dd-trace/src/**/*.js\" -- npm run test:core --  --reporter mocha-multi-reporters --reporter-options configFile=mocha-reporter-config.json",
+    "test:trace:core": "mocha --exit --expose-gc --file packages/dd-trace/test/setup/core.js \"packages/dd-trace/test/**/*.spec.js\"",
+    "test:trace:core:ci": "nyc --include \"packages/dd-trace/src/**/*.js\" -- npm run test:trace:core --  --reporter mocha-multi-reporters --reporter-options configFile=mocha-reporter-config.json",
+    "test:core": "mocha --file packages/datadog-core/test/setup.js 'packages/datadog-core/test/**/*.spec.js'",
+    "test:core:ci": "nyc --include 'packages/datadog-core/src/**/*.js' -- npm run test:core -- --reporter mocha-multi-reporters --reporter-options configFile=mocha-reporter-config.json",
     "test:plugins": "mocha --exit --file \"packages/dd-trace/test/setup/core.js\" \"packages/datadog-plugin-@($(echo $PLUGINS))/test/**/*.spec.js\"",
     "test:plugins:ci": "yarn services && nyc --include \"packages/datadog-plugin-@($(echo $PLUGINS))/src/**/*.js\" -- npm run test:plugins -- --reporter mocha-multi-reporters --reporter-options configFile=mocha-reporter-config.json",
     "test:plugins:upstream": "node ./packages/dd-trace/test/plugins/suite.js",
@@ -56,15 +58,15 @@
     "node": ">=12"
   },
   "dependencies": {
-    "@datadog/native-appsec": "^0.3.1",
-    "@datadog/native-metrics": "^1.0.0",
-    "@datadog/pprof": "^0.1.3",
+    "@datadog/native-appsec": "^0.6.0",
+    "@datadog/native-metrics": "^1.0.1",
+    "@datadog/pprof": "^0.3.0",
     "@datadog/sketches-js": "^1.0.4",
     "@types/node": "^10.12.18",
     "crypto-randomuuid": "^1.0.0",
-    "diagnostics_channel": "^1.0.0",
+    "diagnostics_channel": "^1.1.0",
     "form-data": "^3.0.0",
-    "import-in-the-middle": "^1.1.1",
+    "import-in-the-middle": "^1.1.2",
     "koalas": "^1.0.2",
     "limiter": "^1.1.4",
     "lodash.kebabcase": "^4.1.1",
@@ -77,13 +79,14 @@
     "opentracing": ">=0.12.1",
     "path-to-regexp": "^0.1.2",
     "performance-now": "^2.1.0",
+    "retry": "^0.10.1",
     "semver": "^5.5.0",
     "source-map": "^0.7.3",
     "source-map-resolve": "^0.6.0"
   },
   "devDependencies": {
     "autocannon": "^4.5.2",
-    "axios": "^0.21.1",
+    "axios": "^0.21.2",
     "benchmark": "^2.1.4",
     "body-parser": "^1.18.2",
     "chai": "^4.2.0",
@@ -112,7 +115,6 @@
     "nock": "^11.3.3",
     "nyc": "^15.1.0",
     "proxyquire": "^1.8.0",
-    "retry": "^0.10.1",
     "rimraf": "^3.0.0",
     "sinon": "^11.1.2",
     "sinon-chai": "^3.7.0",

package/packages/datadog-core/index.js

@@ -0,0 +1,7 @@
+'use strict'
+
+const LocalStorage = require('./src/storage')
+
+const storage = new LocalStorage()
+
+module.exports = { storage }

package/packages/datadog-core/src/storage/async_hooks.js

@@ -0,0 +1,49 @@
+'use strict'
+
+const { createHook, executionAsyncId } = require('async_hooks')
+const AsyncResourceStorage = require('./async_resource')
+
+class AsyncHooksStorage extends AsyncResourceStorage {
+  constructor () {
+    super()
+
+    this._resources = new Map()
+  }
+
+  disable () {
+    super.disable()
+
+    this._resources.clear()
+  }
+
+  _createHook () {
+    return createHook({
+      init: this._init.bind(this),
+      destroy: this._destroy.bind(this)
+    })
+  }
+
+  _init (asyncId, type, triggerAsyncId, resource) {
+    super._init.apply(this, arguments)
+
+    this._resources.set(asyncId, resource)
+  }
+
+  _destroy (asyncId) {
+    this._resources.delete(asyncId)
+  }
+
+  _executionAsyncResource () {
+    const asyncId = executionAsyncId()
+
+    let resource = this._resources.get(asyncId)
+
+    if (!resource) {
+      this._resources.set(asyncId, resource = {})
+    }
+
+    return resource
+  }
+}
+
+module.exports = AsyncHooksStorage

package/packages/datadog-core/src/storage/async_resource.js

@@ -0,0 +1,76 @@
+'use strict'
+
+const { createHook, executionAsyncResource } = require('async_hooks')
+
+class AsyncResourceStorage {
+  constructor () {
+    this._ddResourceStore = Symbol('ddResourceStore')
+    this._enabled = false
+    this._hook = this._createHook()
+  }
+
+  disable () {
+    if (!this._enabled) return
+
+    this._hook.disable()
+    this._enabled = false
+  }
+
+  getStore () {
+    if (!this._enabled) return
+
+    const resource = this._executionAsyncResource()
+
+    return resource[this._ddResourceStore]
+  }
+
+  enterWith (store) {
+    this._enable()
+
+    const resource = this._executionAsyncResource()
+
+    resource[this._ddResourceStore] = store
+  }
+
+  run (store, callback, ...args) {
+    this._enable()
+
+    const resource = this._executionAsyncResource()
+    const oldStore = resource[this._ddResourceStore]
+
+    resource[this._ddResourceStore] = store
+
+    try {
+      return callback(...args)
+    } finally {
+      resource[this._ddResourceStore] = oldStore
+    }
+  }
+
+  _createHook () {
+    return createHook({
+      init: this._init.bind(this)
+    })
+  }
+
+  _enable () {
+    if (this._enabled) return
+
+    this._enabled = true
+    this._hook.enable()
+  }
+
+  _init (asyncId, type, triggerAsyncId, resource) {
+    const currentResource = this._executionAsyncResource()
+
+    if (Object.prototype.hasOwnProperty.call(currentResource, this._ddResourceStore)) {
+      resource[this._ddResourceStore] = currentResource[this._ddResourceStore]
+    }
+  }
+
+  _executionAsyncResource () {
+    return executionAsyncResource()
+  }
+}
+
+module.exports = AsyncResourceStorage

package/packages/datadog-core/src/storage/index.js

@@ -0,0 +1,14 @@
+'use strict'
+
+// TODO: default to AsyncLocalStorage when it supports triggerAsyncResource
+
+const semver = require('semver')
+
+// https://github.com/nodejs/node/pull/33801
+const hasJavaScriptAsyncHooks = semver.satisfies(process.versions.node, '>=14.5 || ^12.19.0')
+
+if (hasJavaScriptAsyncHooks) {
+  module.exports = require('./async_resource')
+} else {
+  module.exports = require('./async_hooks')
+}

package/packages/datadog-plugin-cucumber/src/index.js

@@ -1,5 +1,3 @@
-const { relative } = require('path')
-
 const { SAMPLING_RULE_DECISION } = require('../../dd-trace/src/constants')
 
 const {
@@ -7,11 +5,13 @@ const {
   TEST_NAME,
   TEST_SUITE,
   TEST_STATUS,
+  TEST_FRAMEWORK_VERSION,
   TEST_SKIP_REASON,
   CI_APP_ORIGIN,
   ERROR_MESSAGE,
   getTestEnvironmentMetadata,
-  finishAllTraceSpans
+  finishAllTraceSpans,
+  getTestSuitePath
 } = require('../../dd-trace/src/plugins/util/test')
 
 function setStatusFromResult (span, result, tag) {
@@ -42,17 +42,18 @@ function setStatusFromResultLatest (span, result, tag) {
   }
 }
 
-function createWrapRun (tracer, testEnvironmentMetadata, getTestSuiteName, setStatus) {
+function createWrapRun (tracer, testEnvironmentMetadata, sourceRoot, setStatus) {
   return function wrapRun (run) {
     return function handleRun () {
       const testName = this.pickle.name
-      const testSuite = getTestSuiteName(this.pickle.uri)
+      const testSuite = getTestSuitePath(this.pickle.uri, sourceRoot)
 
       const commonSpanTags = {
         [TEST_TYPE]: 'test',
         [TEST_NAME]: testName,
         [TEST_SUITE]: testSuite,
         [SAMPLING_RULE_DECISION]: 1,
+        [TEST_FRAMEWORK_VERSION]: tracer._version,
         ...testEnvironmentMetadata
       }
 
@@ -105,14 +106,11 @@ module.exports = [
     patch (PickleRunner, tracer) {
       const testEnvironmentMetadata = getTestEnvironmentMetadata('cucumber')
       const sourceRoot = process.cwd()
-      const getTestSuiteName = (pickleUri) => {
-        return relative(sourceRoot, pickleUri)
-      }
       const pl = PickleRunner.default
       this.wrap(
         pl.prototype,
         'run',
-        createWrapRun(tracer, testEnvironmentMetadata, getTestSuiteName, setStatusFromResult)
+        createWrapRun(tracer, testEnvironmentMetadata, sourceRoot, setStatusFromResult)
       )
       const getResourceName = (testStep) => {
         return testStep.isHook ? 'hook' : testStep.pickleStep.text
@@ -132,14 +130,11 @@ module.exports = [
     patch (TestCaseRunner, tracer) {
       const testEnvironmentMetadata = getTestEnvironmentMetadata('cucumber')
       const sourceRoot = process.cwd()
-      const getTestSuiteName = (pickleUri) => {
-        return relative(sourceRoot, pickleUri)
-      }
       const pl = TestCaseRunner.default
       this.wrap(
         pl.prototype,
         'run',
-        createWrapRun(tracer, testEnvironmentMetadata, getTestSuiteName, setStatusFromResultLatest)
+        createWrapRun(tracer, testEnvironmentMetadata, sourceRoot, setStatusFromResultLatest)
       )
       const getResourceName = (testStep) => {
         return testStep.text

package/packages/datadog-plugin-cypress/src/plugin.js

@@ -3,6 +3,7 @@ const {
   TEST_NAME,
   TEST_SUITE,
   TEST_STATUS,
+  TEST_FRAMEWORK_VERSION,
   getTestEnvironmentMetadata,
   CI_APP_ORIGIN,
   getTestParentSpan
@@ -19,7 +20,7 @@ const CYPRESS_STATUS_TO_TEST_STATUS = {
   skipped: 'skip'
 }
 
-function getTestSpanMetadata (tracer, testName, testSuite) {
+function getTestSpanMetadata (tracer, testName, testSuite, cypressConfig) {
   const childOf = getTestParentSpan(tracer)
 
   return {
@@ -29,7 +30,8 @@ function getTestSpanMetadata (tracer, testName, testSuite) {
     [TEST_NAME]: testName,
     [TEST_SUITE]: testSuite,
     [SAMPLING_RULE_DECISION]: 1,
-    [SAMPLING_PRIORITY]: AUTO_KEEP
+    [SAMPLING_PRIORITY]: AUTO_KEEP,
+    [TEST_FRAMEWORK_VERSION]: cypressConfig.version
   }
 }
 
@@ -50,7 +52,7 @@ module.exports = (on, config) => {
         childOf,
         resource,
         ...testSpanMetadata
-      } = getTestSpanMetadata(tracer, testName, testSuite)
+      } = getTestSpanMetadata(tracer, testName, testSuite, config)
 
       if (!activeSpan) {
         activeSpan = tracer.startSpan('cypress.test', {

package/packages/datadog-plugin-graphql/src/index.js

@@ -9,20 +9,17 @@ let tools
 function createWrapExecute (tracer, config, defaultFieldResolver) {
   return function wrapExecute (execute) {
     return function executeWithTrace () {
-      const args = normalizeArgs(arguments)
+      const args = normalizeArgs(arguments, tracer, config, defaultFieldResolver)
       const schema = args.schema
       const document = args.document
       const source = document && document._datadog_source
-      const fieldResolver = args.fieldResolver || defaultFieldResolver
-      const contextValue = args.contextValue = args.contextValue || {}
+      const contextValue = args.contextValue
       const operation = getOperation(document, args.operationName)
 
       if (contextValue._datadog_graphql) {
         return execute.apply(this, arguments)
       }
 
-      args.fieldResolver = wrapResolve(fieldResolver, tracer, config)
-
       if (schema) {
         wrapFields(schema._queryType, tracer, config)
         wrapFields(schema._mutationType, tracer, config)
@@ -32,7 +29,7 @@ function createWrapExecute (tracer, config, defaultFieldResolver) {
 
       contextValue._datadog_graphql = { source, span, fields: {} }
 
-      return call(execute, span, this, [args], (err, res) => {
+      return call(execute, span, this, arguments, (err, res) => {
         finishResolvers(contextValue, config)
 
         setError(span, err || (res && res.errors && res.errors[0]))
@@ -212,10 +209,19 @@ function getField (contextValue, path) {
   return contextValue._datadog_graphql.fields[path.join('.')]
 }
 
-function normalizeArgs (args) {
-  if (args.length === 1) {
-    return args[0]
-  }
+function normalizeArgs (args, tracer, config, defaultFieldResolver) {
+  if (args.length !== 1) return normalizePositional(args, tracer, config, defaultFieldResolver)
+
+  args[0].contextValue = args[0].contextValue || {}
+  args[0].fieldResolver = wrapResolve(args[0].fieldResolver || defaultFieldResolver, tracer, config)
+
+  return args[0]
+}
+
+function normalizePositional (args, tracer, config, defaultFieldResolver) {
+  args[3] = args[3] || {} // contextValue
+  args[6] = wrapResolve(args[6] || defaultFieldResolver, tracer, config) // fieldResolver
+  args.length = Math.max(args.length, 7)
 
   return {
     schema: args[0],

package/packages/datadog-plugin-grpc/src/client.js

@@ -48,7 +48,7 @@ function createWrapMakeClientConstructor (tracer, config) {
 function wrapPackageDefinition (tracer, config, def) {
   for (const name in def) {
     if (def[name].format) continue
-    if (def[name].service) {
+    if (def[name].service && def[name].prototype) {
       wrapClientConstructor(tracer, config, def[name], def[name].service)
     } else {
       wrapPackageDefinition(tracer, config, def[name])

package/packages/datadog-plugin-http/src/server.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const web = require('../../dd-trace/src/plugins/util/web')
-const Scope = require('../../dd-trace/src/scope/base')
+const Scope = require('../../dd-trace/src/scope')
 
 function createWrapEmit (tracer, config) {
   config = web.normalizeConfig(config)

package/packages/datadog-plugin-jest/src/jest-environment.js

@@ -5,13 +5,16 @@ const {
   TEST_NAME,
   TEST_SUITE,
   TEST_STATUS,
+  TEST_FRAMEWORK_VERSION,
+  JEST_TEST_RUNNER,
   ERROR_MESSAGE,
   ERROR_TYPE,
   TEST_PARAMETERS,
   CI_APP_ORIGIN,
   getTestEnvironmentMetadata,
   getTestParametersString,
-  finishAllTraceSpans
+  finishAllTraceSpans,
+  getTestSuitePath
 } = require('../../dd-trace/src/plugins/util/test')
 const {
   getFormattedJestTestParameters,
@@ -30,7 +33,7 @@ function wrapEnvironment (BaseEnvironment) {
   return class DatadogJestEnvironment extends BaseEnvironment {
     constructor (config, context) {
       super(config, context)
-      this.testSuite = context.testPath.replace(`${config.rootDir}/`, '')
+      this.testSuite = getTestSuitePath(context.testPath, config.rootDir)
       this.testSpansByTestName = {}
       this.originalTestFnByTestName = {}
     }
@@ -137,7 +140,9 @@ function createHandleTestEvent (tracer, testEnvironmentMetadata, instrumenter) {
     const spanTags = {
       ...commonSpanTags,
       [TEST_NAME]: testName,
-      [TEST_SUITE]: this.testSuite
+      [TEST_SUITE]: this.testSuite,
+      [TEST_FRAMEWORK_VERSION]: tracer._version,
+      [JEST_TEST_RUNNER]: 'jest-circus'
     }
 
     const testParametersString = getTestParametersString(nameToParams, event.test.name)

package/packages/datadog-plugin-jest/src/jest-jasmine2.js

@@ -5,9 +5,12 @@ const {
   TEST_NAME,
   TEST_SUITE,
   TEST_STATUS,
+  TEST_FRAMEWORK_VERSION,
+  JEST_TEST_RUNNER,
   CI_APP_ORIGIN,
   getTestEnvironmentMetadata,
-  finishAllTraceSpans
+  finishAllTraceSpans,
+  getTestSuitePath
 } = require('../../dd-trace/src/plugins/util/test')
 const { getTestSpanTags, setSuppressedErrors } = require('./util')
 
@@ -21,14 +24,19 @@ function createWrapIt (tracer, globalConfig, globalInput, testEnvironmentMetadat
 
       const { childOf, commonSpanTags } = getTestSpanTags(tracer, testEnvironmentMetadata)
 
-      const testSuite = globalInput.jasmine.testPath.replace(`${globalConfig.rootDir}/`, '')
+      const testSuite = getTestSuitePath(globalInput.jasmine.testPath, globalConfig.rootDir)
 
       const newSpecFunction = tracer.wrap(
         'jest.test',
         {
           type: 'test',
           childOf,
-          tags: { ...commonSpanTags, [TEST_SUITE]: testSuite }
+          tags: {
+            ...commonSpanTags,
+            [TEST_SUITE]: testSuite,
+            [TEST_FRAMEWORK_VERSION]: tracer._version,
+            [JEST_TEST_RUNNER]: 'jest-jasmine2'
+          }
         },
         async (done) => {
           const testSpan = tracer.scope().active()
@@ -107,7 +115,7 @@ function createWrapItSkip (tracer, globalConfig, globalInput, testEnvironmentMet
     return function itSkipWithTrace () {
       const { childOf, commonSpanTags } = getTestSpanTags(tracer, testEnvironmentMetadata)
 
-      const testSuite = globalInput.jasmine.testPath.replace(`${globalConfig.rootDir}/`, '')
+      const testSuite = getTestSuitePath(globalInput.jasmine.testPath, globalConfig.rootDir)
 
       const spec = it.apply(this, arguments)
 
@@ -123,7 +131,9 @@ function createWrapItSkip (tracer, globalConfig, globalInput, testEnvironmentMet
             [RESOURCE_NAME]: resource,
             [TEST_NAME]: testName,
             [TEST_SUITE]: testSuite,
-            [TEST_STATUS]: 'skip'
+            [TEST_STATUS]: 'skip',
+            [TEST_FRAMEWORK_VERSION]: tracer._version,
+            [JEST_TEST_RUNNER]: 'jest-jasmine2'
           }
         }
       )

package/packages/datadog-plugin-mocha/src/index.js

@@ -9,34 +9,43 @@ const {
   TEST_SUITE,
   TEST_STATUS,
   TEST_PARAMETERS,
+  TEST_FRAMEWORK_VERSION,
   CI_APP_ORIGIN,
   getTestEnvironmentMetadata,
   getTestParametersString,
   finishAllTraceSpans,
-  getTestParentSpan
+  getTestParentSpan,
+  getTestSuitePath
 } = require('../../dd-trace/src/plugins/util/test')
 
 function getTestSpanMetadata (tracer, test, sourceRoot) {
   const childOf = getTestParentSpan(tracer)
 
-  const { file: testSuite } = test
+  const { file: testSuiteAbsolutePath } = test
   const fullTestName = test.fullTitle()
-  const strippedTestSuite = testSuite ? testSuite.replace(`${sourceRoot}/`, '') : ''
+  const testSuite = getTestSuitePath(testSuiteAbsolutePath, sourceRoot)
 
   return {
     childOf,
-    resource: `${strippedTestSuite}.${fullTestName}`,
+    resource: `${testSuite}.${fullTestName}`,
     [TEST_TYPE]: 'test',
     [TEST_NAME]: fullTestName,
-    [TEST_SUITE]: strippedTestSuite,
+    [TEST_SUITE]: testSuite,
     [SAMPLING_RULE_DECISION]: 1,
-    [SAMPLING_PRIORITY]: AUTO_KEEP
+    [SAMPLING_PRIORITY]: AUTO_KEEP,
+    [TEST_FRAMEWORK_VERSION]: tracer._version
   }
 }
 
 function createWrapRunTest (tracer, testEnvironmentMetadata, sourceRoot) {
   return function wrapRunTest (runTest) {
     return async function runTestWithTrace () {
+      // `runTest` is rerun when retries are configured through `this.retries` and the test fails.
+      // This clause prevents rewrapping `this.test.fn` when it has already been wrapped.
+      if (this.test._currentRetry !== undefined && this.test._currentRetry !== 0) {
+        return runTest.apply(this, arguments)
+      }
+
       let specFunction = this.test.fn
       if (specFunction.length) {
         specFunction = promisify(specFunction)

package/packages/dd-trace/src/appsec/addresses.js

@@ -4,6 +4,8 @@ module.exports = {
   HTTP_INCOMING_URL: 'server.request.uri.raw',
   HTTP_INCOMING_HEADERS: 'server.request.headers.no_cookies',
   HTTP_INCOMING_METHOD: 'server.request.method',
-  HTTP_INCOMING_REMOTE_IP: 'server.request.remote_ip',
-  HTTP_INCOMING_REMOTE_PORT: 'server.request.remote_port'
+  HTTP_INCOMING_REMOTE_IP: 'server.request.client_ip',
+  HTTP_INCOMING_REMOTE_PORT: 'server.request.client_port',
+  HTTP_INCOMING_RESPONSE_CODE: 'server.response.status',
+  HTTP_INCOMING_RESPONSE_HEADERS: 'server.response.headers.no_cookies'
 }