stackify
diff --git a/‎src/Stackify/Log/MessageBuilder.php‎
Lines changed: 11 additions & 3 deletions b/‎src/Stackify/Log/MessageBuilder.php‎
Lines changed: 11 additions & 3 deletions
diff --git a/‎src/Stackify/Log/Monolog/Handler.php‎
Lines changed: 1 addition & 1 deletion b/‎src/Stackify/Log/Monolog/Handler.php‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Stackify/Log/Standalone/Logger.php‎
Lines changed: 1 addition & 1 deletion b/‎src/Stackify/Log/Standalone/Logger.php‎
Lines changed: 1 addition & 1 deletion
@@ -20,14 +20,13 @@ class MessageBuilder
     protected $appName;
     protected $environmentName;
 
-    public function __construct($loggerName, $appName, $environmentName)
+    public function __construct($loggerName, $appName, $environmentName = null)
     {
         if (!function_exists('json_encode')) {
             throw new InitializationException('JSON extension is required for Stackify logger');
         }
-        // @TODO validate $appName and $environmentName ?
         $this->loggerName = $loggerName;
-        $this->appName = $appName;
+        $this->appName = $this->validateNotEmpty('AppName', $appName);
         $this->environmentName = $environmentName;
         // set state for environment details
         EnvironmentDetail::getInstance()->init($appName, $environmentName);
@@ -116,4 +115,13 @@ protected function encodeJSON($data)
         return json_encode($data);
     }
 
+    protected function validateNotEmpty($name, $value)
+    {
+        $result = trim($value);
+        if (empty($result)) {
+            throw new InitializationException("$name cannot be empty");
+        }
+        return $result;
+    }
+
 }
@@ -17,7 +17,7 @@ class Handler extends AbstractHandler
      */
     private $transport;
 
-    public function __construct($appName, $environmentName, TransportInterface $transport = null, $level = Logger::DEBUG, $bubble = true)
+    public function __construct($appName, $environmentName = null, TransportInterface $transport = null, $level = Logger::DEBUG, $bubble = true)
     {
         parent::__construct($level, $bubble);
         $messageBuilder = new MessageBuilder('Stackify Monolog v.1.0', $appName, $environmentName);
 
@@ -16,7 +16,7 @@ class Logger extends AbstractLogger
      */
     private $transport;
 
-    public function __construct($appName, $environmentName, TransportInterface $transport = null)
+    public function __construct($appName, $environmentName = null, TransportInterface $transport = null)
     {
         $messageBuilder = new MessageBuilder('Stackify PHP Logger v.1.0', $appName, $environmentName);
         if (null === $transport) {
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ class Handler extends AbstractHandler`
`17`	`17`	`*/`
`18`	`18`	`private $transport;`
`19`	`19`
`20`		`- public function __construct($appName, $environmentName, TransportInterface $transport = null, $level = Logger::DEBUG, $bubble = true)`
	`20`	`+ public function __construct($appName, $environmentName = null, TransportInterface $transport = null, $level = Logger::DEBUG, $bubble = true)`
`21`	`21`	`{`
`22`	`22`	`parent::__construct($level, $bubble);`
`23`	`23`	`$messageBuilder = new MessageBuilder('Stackify Monolog v.1.0', $appName, $environmentName);`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ class Logger extends AbstractLogger`
`16`	`16`	`*/`
`17`	`17`	`private $transport;`
`18`	`18`
`19`		`- public function __construct($appName, $environmentName, TransportInterface $transport = null)`
	`19`	`+ public function __construct($appName, $environmentName = null, TransportInterface $transport = null)`
`20`	`20`	`{`
`21`	`21`	`$messageBuilder = new MessageBuilder('Stackify PHP Logger v.1.0', $appName, $environmentName);`
`22`	`22`	`if (null === $transport) {`