keymetrics / pm2-io-apm

PM2.io APM for Node.JS
Apache License 2.0
147 stars 40 forks source link

Opencensus postgres plugin issue in client path #246

Closed anujkalka closed 5 years ago

anujkalka commented 5 years ago

Opencensus postgres plugin, located at src/census/plugins/pg.ts has incorrect path to client. The variable which describes the version and client path internalFileList = { '7.x': { 'client': 'client' } } describes client to be present at location 'client'. But in version 7 client file is located at location 'lib/client' As a result pg transaction tracking not working even in simple application

Fix: Changing path to lib/client would fix this problem.

anujkalka commented 5 years ago

UPDATE -- To support pg client 6 and 7,, need to update the same code as internalFileList = { '6 - 7': { 'client': 'lib/client' } }

anujkalka commented 5 years ago

Opened pull request for the fix above

anujkalka commented 5 years ago

PR merged. closing issue now