[Andy Leap (andrewleap-optimizely)]

Summary

• If datafile is provided as bytes, convert to str

The PollingConfigManager provides the datafile as bytes.

Test plan

• test_config.test_to_datafile_from_bytes
• test_optimizely_config.test__get_datafile_from_bytes

Issues

Currently if the datafile is provided as bytes, including using the default PollingConfigManager, it is converted to str incorrectly, resulting in an invalid JSON string being returned from ProjectConfig.to_datafile and OptimizelyConfig.get_datafile.

Steps to replicate:

client = optimizely.Optimizely(sdk_key=sdk_key)
project_config = client.config_manager.get_config()
datafile = project_config.to_datafile()
json.loads(datafile)

[Matjaz Pirnovar (Mat001)]

LGTM

[Alexander Chabo (alexanderchabo)]

Hey Andy Leap (@andrewleap-optimizely), do you know when this possibly is released? :)

Our team is somewhat blocked on this as the datafile is reported as malformed when digested in the javascript-sdk.

[Andy Leap (andrewleap-optimizely)]

Alexander Chabo (@alexanderchabo) we have released this change in v4.1.0