Release v1.1.2

Send name of event tags instead of event ID.
Update URL endpoint to the log server.

Author: onufryk

CHANGELOG

@@ -1,3 +1,9 @@
+-------------------------------------------------------------------------------
+1.1.2
+* Send name of event tags instead of event ID.
+* Update URL endpoint to the log server.
+-------------------------------------------------------------------------------
+
 -------------------------------------------------------------------------------
 1.1.1
 * Gracefully handle empty traffic allocation ranges.

lib/optimizely/event_builder.rb

@@ -1,5 +1,5 @@
 #
-#    Copyright 2016, Optimizely and contributors
+#    Copyright 2016-2017, 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.
@@ -69,8 +69,8 @@ def add_common_params(user_id, attributes)
   end
 
   class EventBuilderV2 < BaseEventBuilder
-    CONVERSION_EVENT_ENDPOINT = 'https://p13nlog.dz.optimizely.com/log/event'
-    IMPRESSION_EVENT_ENDPOINT = 'https://p13nlog.dz.optimizely.com/log/decision'
+    CONVERSION_EVENT_ENDPOINT = 'https://logx.optimizely.com/log/event'
+    IMPRESSION_EVENT_ENDPOINT = 'https://logx.optimizely.com/log/decision'
     POST_HEADERS = { 'Content-Type' => 'application/json' }
 
     def create_impression_event(experiment_key, variation_id, user_id, attributes)
@@ -173,7 +173,7 @@ def add_event_tags(event_tags)
         next if event_tag_value.nil?
 
         event_feature = {
-          'id' => event_tag_key,
+          'name' => event_tag_key,
           'type' => 'custom',
           'value' => event_tag_value,
           'shouldIndex' => false,

lib/optimizely/version.rb

@@ -14,5 +14,5 @@
 #    limitations under the License.
 #
 module Optimizely
-  VERSION = '1.1.1'.freeze
+  VERSION = '1.1.2'.freeze
 end

spec/event_builder_spec.rb

@@ -1,5 +1,5 @@
 #
-#    Copyright 2016, Optimizely and contributors
+#    Copyright 2016-2017, 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.
@@ -35,7 +35,7 @@
     time_now = Time.now
     allow(Time).to receive(:now).and_return(time_now)
 
-    @expected_impression_url = 'https://p13nlog.dz.optimizely.com/log/decision'
+    @expected_impression_url = 'https://logx.optimizely.com/log/decision'
     @expected_impression_params = {
       'visitorId' => 'test_user',
       'timestamp' => (time_now.to_f * 1000).to_i,
@@ -53,7 +53,7 @@
       'userFeatures' => [],
     }
 
-    @expected_conversion_url = 'https://p13nlog.dz.optimizely.com/log/event'
+    @expected_conversion_url = 'https://logx.optimizely.com/log/event'
     @expected_conversion_params = {
       'visitorId' => 'test_user',
       'timestamp' => (time_now.to_f * 1000).to_i,
@@ -130,7 +130,7 @@
     }]
     @expected_conversion_params['eventFeatures'] = [
       {
-        'id' => 'revenue',
+        'name' => 'revenue',
         'type' => 'custom',
         'value' => 4200,
         'shouldIndex' => false
@@ -149,7 +149,7 @@
     @expected_conversion_params['eventMetrics'] = []
     @expected_conversion_params['eventFeatures'] = [
       {
-        'id' => 'revenue',
+        'name' => 'revenue',
         'type' => 'custom',
         'value' => "4200",
         'shouldIndex' => false
@@ -167,7 +167,7 @@
   it 'should create a valid V2 Event when create_conversion_event is called with boolean event tag' do
     @expected_conversion_params['eventFeatures'] = [
       {
-        'id' => 'boolean_tag',
+        'name' => 'boolean_tag',
         'type' => 'custom',
         'value' => false,
         'shouldIndex' => false
@@ -188,7 +188,7 @@
   it 'should create a valid V2 Event when create_conversion_event is called with string event tag' do
     @expected_conversion_params['eventFeatures'] = [
       {
-        'id' => 'string_tag',
+        'name' => 'string_tag',
         'type' => 'custom',
         'value' => 'iamstring',
         'shouldIndex' => false
@@ -208,7 +208,7 @@
   it 'should create a valid V2 Event when create_conversion_event is called with integer event tag' do
     @expected_conversion_params['eventFeatures'] = [
       {
-        'id' => 'integer_tag',
+        'name' => 'integer_tag',
         'type' => 'custom',
         'value' => 42,
         'shouldIndex' => false
@@ -228,7 +228,7 @@
   it 'should create a valid V2 Event when create_conversion_event is called with float event tag' do
     @expected_conversion_params['eventFeatures'] = [
       {
-        'id' => 'float_tag',
+        'name' => 'float_tag',
         'type' => 'custom',
         'value' => 42.01,
         'shouldIndex' => false

spec/project_spec.rb

@@ -1,5 +1,5 @@
 #
-#    Copyright 2016, Optimizely and contributors
+#    Copyright 2016-2017, 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.
@@ -412,8 +412,8 @@ class InvalidErrorHandler; end
   let(:error_handler) { Optimizely::RaiseErrorHandler.new }
   let(:spy_logger) { spy('logger') }
   let(:version) { Optimizely::VERSION }
-  let(:impression_log_url) { 'https://p13nlog.dz.optimizely.com/log/decision' }
-  let(:conversion_log_url) { 'https://p13nlog.dz.optimizely.com/log/event' }
+  let(:impression_log_url) { 'https://logx.optimizely.com/log/decision' }
+  let(:conversion_log_url) { 'https://logx.optimizely.com/log/event' }
   let(:project_instance) { Optimizely::Project.new(config_body_JSON, nil, spy_logger, error_handler) }
   let(:time_now) { Time.now }
   let(:post_headers) { { 'Content-Type' => 'application/json' } }
@@ -711,7 +711,7 @@ class InvalidErrorHandler; end
         'eventEntityId' => '111095',
         'eventFeatures' => [
           {
-            'id' => 'revenue',
+            'name' => 'revenue',
             'type' => 'custom',
             'value' => 42,
             'shouldIndex' => false
@@ -808,7 +808,7 @@ class InvalidErrorHandler; end
         'eventName' => 'test_event',
         'eventFeatures' => [
           {
-            'id' => 'revenue',
+            'name' => 'revenue',
             'type' => 'custom',
             'value' => 42,
             'shouldIndex' => false