Merge branch 'master' into pawel/fix_requester_despatcher

Author: pawels-optimizely

CHANGELOG.MD

@@ -8,6 +8,17 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 Changes that have landed but are not yet released.
 - Support for running full feature tests with the SDK.
 
+## [0.2.0] - September 11th, 2019
+This release of the SDK introduces Feature Management capabilities for running both Feature Rollouts as well as Feature Tests using Optimizely Feature Management.
+
+### New Features
+- Introduces feature variable getters via `GetFeatureVariable*` for parameterizing your feature tests.
+- Introduces the `Track` API call for sending conversion events to Optimizely.
+- The `IsFeatureEnabled` API will now send impression events if there is a feature test attached to the feature being accessed.
+
+### Breaking Changes
+- Vendored packages have been removed in favor of declaring dependencies and their versions using `go.mod`.
+
 ## [0.1.0-beta] - August 23rd, 2019
 This is the initial release of the SDK, which includes support for running Feature Rollouts using Optimizely Feature Management.
 

README.md

@@ -95,14 +95,14 @@ module mymodule
 go 1.12
 
 require (
-	github.com/optimizely/go-sdk v0.1.0
+	github.com/optimizely/go-sdk v0.2.0
 )
 ```
 
 If you are already using `go.mod` in your application you can run the following:
 
 ```
-go mod edit -require github.com/optimizely/go-sdk@v0.1.0
+go mod edit -require github.com/optimizely/go-sdk@v0.2.0
 ```
 
 NOTE:
@@ -247,3 +247,7 @@ License (BSD-3 Clause): https://github.com/google/uuid/blob/master/LICENSE
 testify
 Copyright (c) 2012-2018 Mat Ryer and Tyler Bunnell.
 License (MIT): https://github.com/stretchr/testify/blob/master/LICENSE
+
+json-iterator
+Copyright (c) 2016 json-iterator
+License (MIT): https://github.com/json-iterator/go/blob/master/LICENSE

optimizely/config/polling_manager.go

@@ -53,42 +53,42 @@ type PollingProjectConfigManager struct {
 	exeCtx utils.ExecutionCtx // context used for execution control
 }
 
-func (cm *PollingProjectConfigManager) activate(initialPayload []byte, init bool) {
-
-	update := func() {
-		var e error
-		var code int
-		var payload []byte
-		if init && len(initialPayload) > 0 {
-			payload = initialPayload
-		} else {
-			payload, code, e = cm.requester.Get()
-
-			if e != nil {
-				cmLogger.Error(fmt.Sprintf("request returned with http code=%d", code), e)
-			}
-		}
-
-		projectConfig, err := datafileprojectconfig.NewDatafileProjectConfig(payload)
-		if err != nil {
-			cmLogger.Error("failed to create project config", err)
+// SyncConfig gets current datafile and updates projectConfig
+func (cm *PollingProjectConfigManager) SyncConfig(datafile []byte) {
+	var e error
+	var code int
+	if len(datafile) == 0 {
+		datafile, code, e = cm.requester.Get()
+
+		if e != nil {
+			cmLogger.Error(fmt.Sprintf("request returned with http code=%d", code), e)
 		}
+	}
 
-		cm.configLock.Lock()
-		cm.projectConfig = projectConfig
-		cm.err = err
-		cm.configLock.Unlock()
+	projectConfig, err := datafileprojectconfig.NewDatafileProjectConfig(datafile)
+	if err != nil {
+		cmLogger.Error("failed to create project config", err)
 	}
 
+	// TODO: Compare revision numbers here and set projectConfig only if the revision number has changed
+	cm.configLock.Lock()
+	cm.projectConfig = projectConfig
+	cm.err = err
+	cm.configLock.Unlock()
+}
+
+func (cm *PollingProjectConfigManager) start(initialDatafile []byte, init bool) {
+
 	if init {
-		update()
+		cm.SyncConfig(initialDatafile)
 		return
 	}
+
 	t := time.NewTicker(cm.pollingInterval)
 	for {
 		select {
 		case <-t.C:
-			update()
+			cm.SyncConfig([]byte{})
 		case <-cm.exeCtx.GetContext().Done():
 			cmLogger.Debug("Polling Config Manager Stopped")
 			return
@@ -114,10 +114,10 @@ func NewPollingProjectConfigManagerWithOptions(exeCtx utils.ExecutionCtx, sdkKey
 
 	pollingProjectConfigManager := PollingProjectConfigManager{requester: requester, pollingInterval: pollingInterval, exeCtx: exeCtx}
 
-	pollingProjectConfigManager.activate(options.Datafile, true) // initial poll
+	pollingProjectConfigManager.SyncConfig(options.Datafile) // initial poll
 
 	cmLogger.Debug("Polling Config Manager Initiated")
-	go pollingProjectConfigManager.activate([]byte{}, false)
+	go pollingProjectConfigManager.start([]byte{}, false)
 	return &pollingProjectConfigManager
 }
 

optimizely/version.go

@@ -18,7 +18,7 @@
 package optimizely
 
 // Version is the current version of the client
-const Version = "0.1.0-beta"
+const Version = "0.2.0"
 
 // ClientName is the name of the client
 const ClientName = "go-sdk"