Set cookie path to URL of homepage

wagtail_ab_testing/static/wagtail_ab_testing/js/tracker.js

@@ -40,13 +40,14 @@
             }
 
             // Add this goal page/event into the goals data structure
-            goals[window.wagtailAbTesting.goalPageId] = goals[window.wagtailAbTesting.goalPageId] || {}
+            goals[window.wagtailAbTesting.goalPageId] = goals[window.wagtailAbTesting.goalPageId] || {};
             goals[window.wagtailAbTesting.goalPageId][window.wagtailAbTesting.goalEvent] = goals[window.wagtailAbTesting.goalPageId][window.wagtailAbTesting.goalEvent] || [];
 
             // Check if this user is already a participant in this test
             // We could check the cookie instead, but it's possible that the user has cleared their cookies but not local storage
             if (goals[window.wagtailAbTesting.goalPageId][window.wagtailAbTesting.goalEvent].indexOf(window.wagtailAbTesting.testId) === -1) {
                 var cookieName = 'wagtail-ab-testing_' + window.wagtailAbTesting.testId + '_version';
+                var cookiePath = window.wagtailAbTesting.cookiePath;
                 if (!document.cookie.includes(cookieName)) {
                     fetch(
                         window.wagtailAbTesting.urls.registerParticipant, {
@@ -64,7 +65,7 @@
                             // Put the version into a cookie so that Wagtail continues to serve this version
                             var expires = new Date();
                             expires.setFullYear(expires.getFullYear() + 1);
-                            document.cookie = cookieName + '=' + window.wagtailAbTesting.version + '; expires=' + expires.toUTCString();
+                            document.cookie = cookieName + '=' + window.wagtailAbTesting.version + '; path=' + cookiePath + '; expires=' + expires.toUTCString();
 
                             // Store the test ID against the goal event in the goals data structure
                             // We will use this for knowing when to call the goal reached API later

wagtail_ab_testing/templates/wagtail_ab_testing/script.html

@@ -15,6 +15,7 @@
             version: '{{ version|escapejs }}',
             goalEvent: '{{ test.goal_event|escapejs }}',
             goalPageId: {% if test.goal_page %}{{ test.goal_page.id }}{% else %}null{% endif %},
+            cookiePath: '{{ cookie_path|escapejs }}',
             {% endif %}
 
             urls: {

wagtail_ab_testing/templatetags/wagtail_ab_testing_tags.py

@@ -1,4 +1,5 @@
 from django import template
+from wagtail.models.sites import Site
 
 from wagtail_ab_testing.models import AbTest
 from wagtail_ab_testing.utils import request_is_trackable
@@ -9,11 +10,13 @@
 @register.inclusion_tag('wagtail_ab_testing/script.html', takes_context=True)
 def wagtail_ab_testing_script(context):
     request = context['request']
+    site = Site.find_for_request(request)
     serving_variant = getattr(request, 'wagtail_ab_testing_serving_variant', False)
 
     return {
         'track': request_is_trackable(request),
         'page': context.get('page', None),
         'test': getattr(request, 'wagtail_ab_testing_test', None),
         'version': AbTest.VERSION_VARIANT if serving_variant else AbTest.VERSION_CONTROL,
+        'cookie_path': site.root_page.url,
     }