Merge pull request #20 from bufferapp/task/message-fields

Don't show dd trace hints in local env

Author: erickhun

example.php

@@ -17,7 +17,7 @@
 BuffLog::error("I am an error");
 BuffLog::error("I am an error", ["mean" => "70"]);
 
-BuffLog::criticals("I am criticals information with a typo and you shouldn't see me!");
+BuffLog::critical("I am criticals information with a typo and you shouldn't see me!");
 BuffLog::critical("I am critical information!", ["user" => "betrand"]);
 
 BuffLog::critical("I'm critical log, here some extra fancy informations", 

src/BuffLog/BuffLog.php

@@ -43,8 +43,11 @@ protected static function configureInstance()
         if (class_exists("\DDTrace\GlobalTracer")) {
             self::$hasGlobalTracer = true;
         } else {
-            error_log("Tip #1: Can't find \DDTrace\GlobalTracer class. Did you install the Datadog APM tracer extension? It will allow you to have logs enriched with traces making troubleshooting easier.");
-            error_log("Tip #2: If you run a cli mode service (such as a worker), did you set the DD_TRACE_CLI_ENABLED env variable?");
+            // local envs don't need tracing
+            if (getenv("ENVIRONMENT") !== "local") {
+                error_log("Tip #1: Can't find \DDTrace\GlobalTracer class. Did you install the Datadog APM tracer extension? It will allow you to have logs enriched with traces making troubleshooting easier.");
+                error_log("Tip #2: If you run a cli mode service (such as a worker), did you set the DD_TRACE_CLI_ENABLED env variable?");
+            }
         }
 
         // @TODO: We could potentially use the Kubernetes downward API to