Skip to content

Commit b3e2b28

Browse files
fix: correct error returned when SDK key is invalid (#203)
* fix: correct error returned when SDK key is invalid
1 parent da047e2 commit b3e2b28

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

pkg/client/client.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -400,7 +400,6 @@ func (o *OptimizelyClient) getExperimentDecision(experimentKey string, userConte
400400

401401
projectConfig, e := o.GetProjectConfig()
402402
if e != nil {
403-
logger.Error("Error calling getExperimentDecision", e)
404403
return decisionContext, experimentDecision, e
405404
}
406405

pkg/config/polling_manager.go

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ import (
2929
"github.com/optimizely/go-sdk/pkg/notification"
3030
"github.com/optimizely/go-sdk/pkg/registry"
3131
"github.com/optimizely/go-sdk/pkg/utils"
32+
33+
"github.com/pkg/errors"
3234
)
3335

3436
// DefaultPollingInterval sets default interval for polling manager
@@ -112,9 +114,10 @@ func (cm *PollingProjectConfigManager) SyncConfig(sdkKey string, datafile []byte
112114
}
113115

114116
if e != nil {
115-
cmLogger.Error(fmt.Sprintf("request returned with http code=%d", code), e)
117+
msg := "unable to fetch fresh datafile"
118+
cmLogger.Warning(msg)
116119
cm.configLock.Lock()
117-
closeMutex(e)
120+
closeMutex(errors.New(fmt.Sprintf("%s, reason (http status code): %s", msg, e.Error())))
118121
return
119122
}
120123

@@ -137,7 +140,7 @@ func (cm *PollingProjectConfigManager) SyncConfig(sdkKey string, datafile []byte
137140
cm.configLock.Lock()
138141

139142
if err != nil {
140-
cmLogger.Error("failed to create project config", err)
143+
cmLogger.Warning("failed to create project config")
141144
closeMutex(err)
142145
return
143146
}

0 commit comments

Comments
 (0)