fix: Correct sampleRate behaviour (#1627)

Author: kamilogorek

CHANGELOG.md

@@ -11,6 +11,8 @@
 - [node] feat: Transactions handling for RequestHandler in Express/Hapi
 - [node] feat: Allow requestHandler to be configured
 - [node] feat: Make node transactions a pluggable integration with tests 
+- [core] feat: getRequestheaders should handle legacy DSNs
+- [core] fix: correct sampleRate behaviour
 
 ## 4.0.6
 

packages/core/src/baseclient.ts

@@ -347,7 +347,9 @@ export abstract class BaseClient<B extends Backend, O extends Options> implement
 
     const { beforeSend, sampleRate } = this.getOptions();
 
-    if (typeof sampleRate === 'number' && sampleRate > Math.random()) {
+    // 1.0 === 100% events are sent
+    // 0.0 === 0% events are sent
+    if (typeof sampleRate === 'number' && Math.random() > sampleRate) {
       return {
         status: Status.Skipped,
       };