Skip to content

Commit

Permalink
Merge pull request #122 from l3mpire/disable-instrumentation2
Browse files Browse the repository at this point in the history
Add disableInstrumentation option
  • Loading branch information
zodern authored Nov 20, 2024
2 parents 7ec44f7 + 6d14636 commit 01314b8
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,8 @@ You should use the same method that you used to give the agent the app id and se
| disableNtp | OPTIONS_DISABLE_NTP | false | Disable NTP time synchronization used to get the accurate time in case the server or client's clock is wrong |
| stalledTimeout | STALLED_TIMEOUT | 1800000 (30m) | Timeout used to detect when methods and subscriptions might be stalled (have been running for a long time and might never return). The value is in milliseconds, and can be disabled by setting it to 0 |
| proxy | MONTI_OPTIONS_PROXY | none | Allows you to connect to Monti APM using a proxy |
| disableInstrumentation | MONTI_DISABLE_INSTRUMENTATION | false | Disables instrumentation. Useful for disabling without uninstalling the package |


### Traces

Expand Down
4 changes: 4 additions & 0 deletions lib/hijack/instrument.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@ import { wrapRedisOplogObserveDriver } from './redis_oplog';

let instrumented = false;
Kadira._startInstrumenting = function (callback) {
if (Meteor.settings?.monti?.disableInstrumentation || process.env.MONTI_DISABLE_INSTRUMENTATION) {
console.log('Monti APM: Instrumentation is disabled.');

Check failure on line 24 in lib/hijack/instrument.js

View workflow job for this annotation

GitHub Actions / build

Unexpected console statement
return;
}
if (instrumented) {
callback();
return;
Expand Down

0 comments on commit 01314b8

Please sign in to comment.