search

HappyPorch / uSplit

A/B Testing plugin for Umbraco
The Unlicense
11 stars 7 forks source link

uSplit throws exception if any experiment in Analytics account contains a description #15

Closed hakon-knowit closed 7 years ago

hakon-knowit commented 7 years ago

Steps to reproduce: 1) Create an experiment in Google Analytics (not from uSplit) 2) Write anything in the experiment description/notes (step 4). I've used ☕. 3) Save the experiment and open A/B testing in backoffice:

Error in backoffice

Stack trace:

2017-07-03 16:41:10,500 [11] ERROR Endzone.uSplit.Pipeline.ExperimentsPipeline - Exception has been thrown when uSplit processed the request
Newtonsoft.Json.JsonReaderException: Unexpected character encountered while parsing value: ☕. Path '', line 0, position 0.
   at Umbraco.Core.Cache.HttpRuntimeCacheProvider.GetCacheItem(String cacheKey, Func`1 getCacheItem, Nullable`1 timeout, Boolean isSliding, CacheItemPriority priority, CacheItemRemovedCallback removedCallback, CacheDependency dependency)
   at Umbraco.Core.Cache.HttpRuntimeCacheProvider.GetCacheItem(String cacheKey, Func`1 getCacheItem, Nullable`1 timeout, Boolean isSliding, CacheItemPriority priority, CacheItemRemovedCallback removedCallback, String[] dependentFiles)
   at Umbraco.Core.Cache.DeepCloneRuntimeCacheProvider.GetCacheItem(String cacheKey, Func`1 getCacheItem, Nullable`1 timeout, Boolean isSliding, CacheItemPriority priority, CacheItemRemovedCallback removedCallback, String[] dependentFiles)
   at Umbraco.Core.Cache.CacheProviderExtensions.GetCacheItem[T](IRuntimeCacheProvider provider, String cacheKey, Func`1 getCacheItem, Nullable`1 timeout, Boolean isSliding, CacheItemPriority priority, CacheItemRemovedCallback removedCallback, String[] dependentFiles)
   at Endzone.uSplit.Commands.GetApplicableCachedExperiments.ExecuteAsync()
   at Endzone.uSplit.Pipeline.ExperimentsPipeline.Process(PublishedContentRequest request)
   at Endzone.uSplit.Pipeline.ExperimentsPipeline.PublishedContentRequestOnPrepared(Object sender, EventArgs eventArgs)