Delay checking null values of language and endpoint attributes until building. This allows that if null values are set in the builder the default ones wil l be used

basoko · basoko · commit e50d17bd0b3a · 2018-05-22T18:46:36.000+02:00
diff --git a/rollbar-java/src/main/java/com/rollbar/notifier/config/ConfigBuilder.java b/rollbar-java/src/main/java/com/rollbar/notifier/config/ConfigBuilder.java
@@ -75,11 +75,8 @@ public class ConfigBuilder {
   private ConfigBuilder(String accessToken) {
     // Defaults
     this.accessToken = accessToken;
-    this.language = "java";
     this.handleUncaughtErrors = true;
     this.enabled = true;
-    this.endpoint = "https://api.rollbar.com/api/1/item/";
-
   }
 
   /**
@@ -375,6 +372,12 @@ public ConfigBuilder enabled(boolean enabled) {
    * @return the config.
    */
   public Config build() {
+    if (this.language == null) {
+      this.language = "java";
+    }
+    if (this.endpoint == null) {
+      this.endpoint = SyncSender.DEFAULT_API_ENDPOINT;
+    }
     if (this.notifier == null) {
       this.notifier = new NotifierProvider();
     }
