Add proper header

Author: jpomykala

src/main/java/io/simplelocalize/cli/SimplelocalizeCliCommand.java

@@ -85,9 +85,9 @@ public void extract(
       {
         configuration.setSearchDir(searchDirectory);
       }
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateExtractConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       ExtractCommand extractCommand = new ExtractCommand(client, configuration);
       extractCommand.invoke();
@@ -196,9 +196,9 @@ public void upload(
         configuration.setDryRun(true);
       }
 
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateUploadConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       UploadCommand uploadCommand = new UploadCommand(client, configuration);
       uploadCommand.invoke();
@@ -263,9 +263,9 @@ public void download(
         configuration.setDownloadOptions(downloadOptions);
       }
 
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateDownloadConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       DownloadCommand downloadCommand = new DownloadCommand(client, configuration);
       downloadCommand.invoke();
@@ -315,9 +315,9 @@ public void pull(
         configuration.setFilterRegex(filterRegex);
       }
 
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateHostingPullConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       PullHostingCommand command = new PullHostingCommand(client, configuration);
       command.invoke();
@@ -357,9 +357,9 @@ public void startAutoTranslation(
         configuration.setAutoTranslation(autoTranslationConfiguration);
       }
 
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateAutoTranslationConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       AutoTranslationCommand command = new AutoTranslationCommand(client, configuration);
       command.invoke();
@@ -406,9 +406,9 @@ public void status(
         configuration.setApiKey(apiKey);
       }
 
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateGetStatusConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       StatusCommand command = new StatusCommand(client);
       command.invoke();
@@ -445,9 +445,9 @@ public void publish(
         configuration.setEnvironment(environment);
       }
 
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateHostingPublishConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       PublishHostingCommand command = new PublishHostingCommand(client, configuration);
       command.invoke();
@@ -480,9 +480,9 @@ public void purge(
         configuration.setApiKey(apiKey);
       }
 
+      this.effectiveCommandConfiguration = configuration;
       ConfigurationValidator configurationValidator = new ConfigurationValidator();
       configurationValidator.validateGetPurgeConfiguration(configuration);
-      this.effectiveCommandConfiguration = configuration;
       SimpleLocalizeClient client = SimpleLocalizeClient.create(configuration.getBaseUrl(), configuration.getApiKey());
       PurgeCommand command = new PurgeCommand(client, force);
       command.invoke();
@@ -512,7 +512,7 @@ private void trySendException(Exception exception)
       client.sendException(effectiveCommandConfiguration, exception);
     } catch (Exception ex)
     {
-      log.error("Unable to send exception to SimpleLocalize, please contact us at [email protected]", ex);
+      log.error("Unable to send exception to SimpleLocalize, please contact us at [email protected]. {}", ex.getMessage());
       if (ex instanceof InterruptedException)
       {
         Thread.currentThread().interrupt();

src/test/java/io/simplelocalize/cli/SimplelocalizeCliCommandTest.java

@@ -208,6 +208,15 @@ void pull() throws IOException
     });
   }
 
+  @Test
+  void sendException() throws IOException
+  {
+    //when
+    assertDoesNotThrow(() -> {
+      sut.pull("my-api-key", "./my-path", null, null, MOCK_SERVER_BASE_URL);
+    });
+  }
+
   @Test
   void status() throws IOException
   {