Noman/temp doc check

.travis.yml

@@ -24,3 +24,8 @@ branches:
   - /^\d+\.\d+\.\d+(-SNAPSHOT|-alpha|-beta)?\d*$/ # trigger builds on tags which are semantically versioned to ship the SDK.
 after_success:
   - ./gradlew coveralls uploadArchives --console plain
+  - ./push-javadoc-to-gh-pages.sh
+env:
+    global:
+    - secure: AscVURpguRPEklN23Y3XQ4mN6KqwW36DJsBMtVB9mxrKxcFlQu9kxNTO0OLoIBzuoKRVnQwxFWIxq0unSeQwRjfgpxS8mfe6UUutF8Zksj3YhAyNwlIG1DP38MDH5iX0NFEjARCfZgBqR0Uq6cS9i+vZLaTvITyQTkOEbrmvX1wn6xfk6F+h+k6k7mDsDwDx6DLluhGGVrJtnKJSd7WWOJfSQk7Qu6zyNs8mvkhueA2HxqB2gIjZwRZFSAsKOL4ZmkgWj46DsK6lkbwArCYwBN59d24A8ZE/OUq9ZkjU/NcWHqmDA93i/wXIBoaTXCMhZG+eJiswcTFSsWhbLVNxzllelr2UgdIzvQ10+L+iCCQoX0x+BHPybfdSRxDxETjd0bv8vh7CE9vNc1ZMT1gh3gLyjjBS1PGw8VmEvjlqV2tPzGZTdO/xD+3ENQBETfAJ8Vum43lNoyoqS9Svpe53coXUSh5J6I9ufnh477UHYB+tJ40hrklLV3puVvWHv/WdqHBfxpBj8ucom/OI9IpqxoNn5OadiqvueBVijE+UiOzv1nvR1j46H2Eqbe8RXnCeCdsX+UqpcnCTiXVYtNxvBCjD57B+cW6IHJf69JnY9i8lEc0RdZI8Fv7KuAN8yPfiZrKFrnO5x1NeYXa2z3g0wi2CWIbIJlf+cMcVprrpsFg=
+    - secure: MdW7OyX7H3MjckrhEhuj4M97r7P1e2egg0UXzvG4t713rpXA9xGXMofdgsDx47/Tg5LlZYPyTunYGAPL8KdS/vXgNEQN9/ERIGl1cRl0bPT2o/+LfR2zqM5PjDDpReCpYLLKmsuU0Oo3W+Lf6ZjC/QuZX/z7X7d3SvZ9o+Aydlqij9KmMNBA6wxSMA5g0uX4Uc5BRpKfnrxOcpoJX2HUkxcSkjV73Hdzcg32/kEJRmPw2XBLcl6NXeNSA0OZKl/4tGJl3w66saseeQg3u66CUvMc5RhmRuNdobp2GrJ+8cRLcIop/KcSM8EnM4SpQdaspWynkeIk/SFfZCoDcOhWFcNWcmxUtNvu/SJPFvQiCHSqG3RjHl5s/r6BZQYfmY/DtMlquUJIgqWCfG69i6VTtNek3dcJDHkGb47yuaAdEVF3qpVALUFWSa26wNmjoMmwXJngOkazXWJs5nVGVXE+8DxBT8nO5Hxc5wMVSJeZAUIB4IzAQOzKiE0Z6DJ5//iJ+9FnEUuObmvHoh593NePzoI+aHzla+wQ6QwKQa3gzhHloN5k4vmuN2I2c+fKwNsklD3TjTw1odpRw1ggHuCC9U+oKQmG3KMGtKynvqo/qfMoPXuLgI69GQDTnEE96xB/zfS2D6tmafQ/ZYjyiFD/EHizJv2A2XMFFTQMpQLJvxo=

core-api/src/main/java/com/optimizely/ab/Optimizely.java

@@ -37,6 +37,7 @@
 import com.optimizely.ab.event.internal.BuildVersionInfo;
 import com.optimizely.ab.event.internal.EventBuilder;
 import com.optimizely.ab.event.internal.payload.EventBatch.ClientEngine;
+import com.optimizely.ab.internal.ControlAttribute;
 import com.optimizely.ab.notification.NotificationCenter;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -749,7 +750,7 @@ public UserProfileService getUserProfileService() {
      *
      * @param projectConfig the current project config
      * @param attributes the attributes map to validate and potentially filter. The reserved key for bucketing id
-     * {@link DecisionService#BUCKETING_ATTRIBUTE} is kept.
+     * {@link DecisionService#{ControlAttribute.BUCKETING_ATTRIBUTE} is kept.
      * @return the filtered attributes map (containing only attributes that are present in the project config) or an
      * empty map if a null attributes object is passed in
      */
@@ -765,7 +766,8 @@ private Map<String, String> filterAttributes(@Nonnull ProjectConfig projectConfi
         Map<String, Attribute> attributeKeyMapping = projectConfig.getAttributeKeyMapping();
         for (Map.Entry<String, String> attribute : attributes.entrySet()) {
             if (!attributeKeyMapping.containsKey(attribute.getKey()) &&
-                    attribute.getKey() != com.optimizely.ab.bucketing.DecisionService.BUCKETING_ATTRIBUTE) {
+                    !attribute.getKey().equals(ControlAttribute.BUCKETING_ATTRIBUTE.toString()) &&
+                        !attribute.getKey().equals(ControlAttribute.USER_AGENT_ATTRIBUTE.toString())) {
                 if (unknownAttributes == null) {
                     unknownAttributes = new ArrayList<String>();
                 }

core-api/src/main/java/com/optimizely/ab/bucketing/DecisionService.java

@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright 2017, Optimizely, Inc. and contributors                        *
+ * Copyright 2017-2018, Optimizely, Inc. and contributors                        *
  *                                                                          *
  * Licensed under the Apache License, Version 2.0 (the "License");          *
  * you may not use this file except in compliance with the License.         *
@@ -25,6 +25,7 @@
 import com.optimizely.ab.error.ErrorHandler;
 import com.optimizely.ab.internal.ExperimentUtils;
 
+import com.optimizely.ab.internal.ControlAttribute;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -46,7 +47,6 @@
  */
 public class DecisionService {
 
-    public static final String BUCKETING_ATTRIBUTE = "$opt_bucketing_id";
     private final Bucketer bucketer;
     private final ErrorHandler errorHandler;
     private final ProjectConfig projectConfig;
@@ -130,8 +130,8 @@ public DecisionService(@Nonnull Bucketer bucketer,
 
         if (ExperimentUtils.isUserInExperiment(projectConfig, experiment, filteredAttributes)) {
             String bucketingId = userId;
-            if (filteredAttributes.containsKey(BUCKETING_ATTRIBUTE)) {
-                bucketingId = filteredAttributes.get(BUCKETING_ATTRIBUTE);
+            if (filteredAttributes.containsKey(ControlAttribute.BUCKETING_ATTRIBUTE.toString())) {
+                bucketingId = filteredAttributes.get(ControlAttribute.BUCKETING_ATTRIBUTE.toString());
             }
             variation = bucketer.bucket(experiment, bucketingId);
 
@@ -211,8 +211,8 @@ public DecisionService(@Nonnull Bucketer bucketer,
         // for all rules before the everyone else rule
         int rolloutRulesLength = rollout.getExperiments().size();
         String bucketingId = userId;
-        if (filteredAttributes.containsKey(BUCKETING_ATTRIBUTE)) {
-            bucketingId = filteredAttributes.get(BUCKETING_ATTRIBUTE);
+        if (filteredAttributes.containsKey(ControlAttribute.BUCKETING_ATTRIBUTE.toString())) {
+            bucketingId = filteredAttributes.get(ControlAttribute.BUCKETING_ATTRIBUTE.toString());
         }
         Variation variation;
         for (int i = 0; i < rolloutRulesLength - 1; i++) {

core-api/src/main/java/com/optimizely/ab/config/ProjectConfig.java

@@ -1,6 +1,6 @@
 /**
  *
- *    Copyright 2016-2017, Optimizely and contributors
+ *    Copyright 2016-2018, Optimizely and contributors
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -24,6 +24,7 @@
 import com.optimizely.ab.error.ErrorHandler;
 import com.optimizely.ab.error.NoOpErrorHandler;
 import com.optimizely.ab.error.RaiseExceptionErrorHandler;
+import com.optimizely.ab.internal.ControlAttribute;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -73,6 +74,7 @@ public String toString() {
     private final String revision;
     private final String version;
     private final boolean anonymizeIP;
+    private final Boolean botFiltering;
     private final List<Attribute> attributes;
     private final List<Audience> audiences;
     private final List<EventType> events;
@@ -100,6 +102,8 @@ public String toString() {
     private final Map<String, Map<String, LiveVariableUsageInstance>> variationToLiveVariableUsageInstanceMapping;
     private final Map<String, Experiment> variationIdToExperimentMapping;
 
+    private final static String RESERVED_ATTRIBUTE_PREFIX = "$opt_";
+
     /**
      *  Forced variations supersede any other mappings.  They are transient and are not persistent or part of
      * the actual datafile. This contains all the forced variations
@@ -123,6 +127,7 @@ public ProjectConfig(String accountId, String projectId, String version, String
         this(
                 accountId,
                 anonymizeIP,
+                null,
                 projectId,
                 revision,
                 version,
@@ -140,6 +145,7 @@ public ProjectConfig(String accountId, String projectId, String version, String
     // v4 constructor
     public ProjectConfig(String accountId,
                          boolean anonymizeIP,
+                         Boolean botFiltering,
                          String projectId,
                          String revision,
                          String version,
@@ -157,6 +163,7 @@ public ProjectConfig(String accountId,
         this.version = version;
         this.revision = revision;
         this.anonymizeIP = anonymizeIP;
+        this.botFiltering = botFiltering;
 
         this.attributes = Collections.unmodifiableList(attributes);
         this.audiences = Collections.unmodifiableList(audiences);
@@ -285,6 +292,35 @@ private List<Experiment> aggregateGroupExperiments(List<Group> groups) {
         return groupExperiments;
     }
 
+    /**
+     * Checks is attributeKey is reserved or not and if it exist in attributeKeyMapping
+     * @param attributeKey
+     * @return AttributeId corresponding to AttributeKeyMapping, AttributeKey when it's a reserved attribute and
+     * null when attributeKey is equal to BOT_FILTERING_ATTRIBUTE key.
+     */
+    public String getAttributeId(ProjectConfig projectConfig, String attributeKey) {
+        String attributeIdOrKey = null;
+        if (!attributeKey.equals(ControlAttribute.BOT_FILTERING_ATTRIBUTE.toString())) {
+            com.optimizely.ab.config.Attribute attribute = projectConfig.getAttributeKeyMapping().get(attributeKey);
+            boolean hasReservedPrefix = attributeKey.startsWith(RESERVED_ATTRIBUTE_PREFIX);
+            if (attribute != null) {
+                if (hasReservedPrefix) {
+                    logger.warn("Attribute {} unexpectedly has reserved prefix {}; using attribute ID instead of reserved attribute name.",
+                            attributeKey, RESERVED_ATTRIBUTE_PREFIX);
+                }
+                attributeIdOrKey = attribute.getId();
+            } else if (hasReservedPrefix) {
+                attributeIdOrKey = attributeKey;
+            } else {
+                logger.debug("Unrecognized Attribute \"{}\"", attributeKey);
+            }
+        } else {
+            logger.warn("Attribute {} unexpectedly has reserved key {}.",
+                    attributeKey, ControlAttribute.BOT_FILTERING_ATTRIBUTE.toString());
+        }
+        return attributeIdOrKey;
+    }
+
     public String getAccountId() {
         return accountId;
     }
@@ -305,6 +341,10 @@ public boolean getAnonymizeIP() {
         return anonymizeIP;
     }
 
+    public Boolean getBotFiltering() {
+        return botFiltering;
+    }
+
     public List<Group> getGroups() {
         return groups;
     }
@@ -540,6 +580,7 @@ public String toString() {
                 ", revision='" + revision + '\'' +
                 ", version='" + version + '\'' +
                 ", anonymizeIP=" + anonymizeIP +
+                ", botFiltering=" + botFiltering +
                 ", attributes=" + attributes +
                 ", audiences=" + audiences +
                 ", events=" + events +

core-api/src/main/java/com/optimizely/ab/config/parser/JsonConfigParser.java

@@ -1,6 +1,6 @@
 /**
  *
- *    Copyright 2016-2017, Optimizely and contributors
+ *    Copyright 2016-2018, Optimizely and contributors
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -81,14 +81,18 @@ public ProjectConfig parseProjectConfig(@Nonnull String json) throws ConfigParse
 
             List<FeatureFlag> featureFlags = null;
             List<Rollout> rollouts = null;
+            Boolean botFiltering = null;
             if (datafileVersion >= Integer.parseInt(ProjectConfig.Version.V4.toString())) {
                 featureFlags = parseFeatureFlags(rootObject.getJSONArray("featureFlags"));
                 rollouts = parseRollouts(rootObject.getJSONArray("rollouts"));
+                if(rootObject.has("botFiltering"))
+                    botFiltering = rootObject.getBoolean("botFiltering");
             }
 
             return new ProjectConfig(
                     accountId,
                     anonymizeIP,
+                    botFiltering,
                     projectId,
                     revision,
                     version,

core-api/src/main/java/com/optimizely/ab/config/parser/JsonSimpleConfigParser.java

@@ -1,6 +1,6 @@
 /**
  *
- *    Copyright 2016-2017, Optimizely and contributors
+ *    Copyright 2016-2018, Optimizely and contributors
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -83,14 +83,18 @@ public ProjectConfig parseProjectConfig(@Nonnull String json) throws ConfigParse
 
             List<FeatureFlag> featureFlags = null;
             List<Rollout> rollouts = null;
+            Boolean botFiltering = null;
             if (datafileVersion >= Integer.parseInt(ProjectConfig.Version.V4.toString())) {
                 featureFlags = parseFeatureFlags((JSONArray) rootObject.get("featureFlags"));
                 rollouts = parseRollouts((JSONArray) rootObject.get("rollouts"));
+                if(rootObject.containsKey("botFiltering"))
+                    botFiltering = (Boolean) rootObject.get("botFiltering");
             }
 
             return new ProjectConfig(
                     accountId,
                     anonymizeIP,
+                    botFiltering,
                     projectId,
                     revision,
                     version,
@@ -103,6 +107,8 @@ public ProjectConfig parseProjectConfig(@Nonnull String json) throws ConfigParse
                     liveVariables,
                     rollouts
             );
+        } catch (RuntimeException ex){
+            throw new ConfigParseException("Unable to parse datafile: " + json, ex);
         } catch (Exception e) {
             throw new ConfigParseException("Unable to parse datafile: " + json, e);
         }

core-api/src/main/java/com/optimizely/ab/config/parser/ProjectConfigGsonDeserializer.java

@@ -1,6 +1,6 @@
 /**
  *
- *    Copyright 2016-2017, Optimizely and contributors
+ *    Copyright 2016-2018, Optimizely and contributors
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -82,16 +82,20 @@ public ProjectConfig deserialize(JsonElement json, Type typeOfT, JsonDeserializa
 
         List<FeatureFlag> featureFlags = null;
         List<Rollout> rollouts = null;
+        Boolean botFiltering = null;
         if (datafileVersion >= Integer.parseInt(ProjectConfig.Version.V4.toString())) {
             Type featureFlagsType = new TypeToken<List<FeatureFlag>>() {}.getType();
             featureFlags = context.deserialize(jsonObject.getAsJsonArray("featureFlags"), featureFlagsType);
             Type rolloutsType = new TypeToken<List<Rollout>>() {}.getType();
             rollouts = context.deserialize(jsonObject.get("rollouts").getAsJsonArray(), rolloutsType);
+            if(jsonObject.has("botFiltering"))
+                botFiltering = jsonObject.get("botFiltering").getAsBoolean();
         }
 
         return new ProjectConfig(
                 accountId,
                 anonymizeIP,
+                botFiltering,
                 projectId,
                 revision,
                 version,

core-api/src/main/java/com/optimizely/ab/config/parser/ProjectConfigJacksonDeserializer.java

@@ -1,6 +1,6 @@
 /**
  *
- *    Copyright 2016-2017, Optimizely and contributors
+ *    Copyright 2016-2018, Optimizely and contributors
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -76,16 +76,20 @@ public ProjectConfig deserialize(JsonParser parser, DeserializationContext conte
 
         List<FeatureFlag> featureFlags = null;
         List<Rollout> rollouts = null;
+        Boolean botFiltering = null;
         if (datafileVersion >= Integer.parseInt(ProjectConfig.Version.V4.toString())) {
             featureFlags = mapper.readValue(node.get("featureFlags").toString(),
                    new TypeReference<List<FeatureFlag>>() {});
             rollouts = mapper.readValue(node.get("rollouts").toString(),
                     new TypeReference<List<Rollout>>(){});
+            if (node.hasNonNull("botFiltering"))
+                botFiltering = node.get("botFiltering").asBoolean();
         }
 
         return new ProjectConfig(
                 accountId,
                 anonymizeIP,
+                botFiltering,
                 projectId,
                 revision,
                 version,

core-api/src/main/java/com/optimizely/ab/event/internal/EventBuilder.java

@@ -17,7 +17,6 @@
 package com.optimizely.ab.event.internal;
 
 import com.optimizely.ab.annotations.VisibleForTesting;
-import com.optimizely.ab.bucketing.DecisionService;
 import com.optimizely.ab.config.EventType;
 import com.optimizely.ab.config.Experiment;
 import com.optimizely.ab.config.ProjectConfig;
@@ -32,6 +31,7 @@
 import com.optimizely.ab.event.internal.serializer.DefaultJsonSerializer;
 import com.optimizely.ab.event.internal.serializer.Serializer;
 import com.optimizely.ab.internal.EventTagUtils;
+import com.optimizely.ab.internal.ControlAttribute;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import javax.annotation.Nonnull;
@@ -44,7 +44,6 @@
 
 public class EventBuilder {
     private static final Logger logger = LoggerFactory.getLogger(EventBuilder.class);
-    static final String ATTRIBUTE_KEY_FOR_BUCKETING_ATTRIBUTE = "optimizely_bucketing_id";
     static final String EVENT_ENDPOINT = "https://logx.optimizely.com/v1/events";
     static final String  ACTIVATE_EVENT_KEY = "campaign_activated";
 
@@ -118,17 +117,25 @@ public LogEvent createConversionEvent(@Nonnull ProjectConfig projectConfig,
     private List<Attribute> buildAttributeList(ProjectConfig projectConfig, Map<String, String> attributes) {
         List<Attribute> attributesList = new ArrayList<Attribute>();
 
-        Map<String, com.optimizely.ab.config.Attribute> attributeMap = projectConfig.getAttributeKeyMapping();
         for (Map.Entry<String, String> entry : attributes.entrySet()) {
-            com.optimizely.ab.config.Attribute projectAttribute = attributeMap.get(entry.getKey());
-            Attribute attribute = new Attribute((projectAttribute != null ? projectAttribute.getId() : null),
-                    entry.getKey(), Attribute.CUSTOM_ATTRIBUTE_TYPE, entry.getValue());
-
-            if (entry.getKey() == DecisionService.BUCKETING_ATTRIBUTE) {
-                attribute = new Attribute(com.optimizely.ab.bucketing.DecisionService.BUCKETING_ATTRIBUTE,
-                        ATTRIBUTE_KEY_FOR_BUCKETING_ATTRIBUTE, Attribute.CUSTOM_ATTRIBUTE_TYPE, entry.getValue());
+            String attributeId = projectConfig.getAttributeId(projectConfig, entry.getKey());
+            if(attributeId != null) {
+                Attribute attribute = new Attribute(attributeId,
+                        entry.getKey(),
+                        Attribute.CUSTOM_ATTRIBUTE_TYPE,
+                        entry.getValue());
+                attributesList.add(attribute);
             }
+        }
 
+        //checks if botFiltering value is not set in the project config file.
+        if(projectConfig.getBotFiltering() != null) {
+            Attribute attribute = new Attribute(
+                    ControlAttribute.BOT_FILTERING_ATTRIBUTE.toString(),
+                    ControlAttribute.BOT_FILTERING_ATTRIBUTE.toString(),
+                    Attribute.CUSTOM_ATTRIBUTE_TYPE,
+                    Boolean.toString(projectConfig.getBotFiltering())
+            );
             attributesList.add(attribute);
         }