FIX: ensure authentication_data cookie is deleted

mozilla · Sep 5, 2019 · c419042 · c419042
1 parent 2b15640
commit c419042
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/app/controllers/mozilla_iam/dinopark_link_controller.rb b/app/controllers/mozilla_iam/dinopark_link_controller.rb
@@ -9,11 +9,13 @@ def link
       profile = Profile.for(current_user)
       profile.dinopark_enabled = true
       profile.force_refresh
+      cookies.delete(:authentication_data)
       render json: { success: true }, status: 200
     end
 
     def dont_show
       Profile.set(current_user, :never_show_dinopark_modal, true)
+      cookies.delete(:authentication_data)
       render json: { success: true }, status: 200
     end
 

diff --git a/assets/javascripts/discourse/initializers/dinopark-log-in.js.es6 b/assets/javascripts/discourse/initializers/dinopark-log-in.js.es6
@@ -7,7 +7,7 @@ export default {
     withPluginApi('0.8.30', api => {
 
       var data = $.cookie("authentication_data")
-      $.removeCookie("authentication_data")
+      $.removeCookie("authentication_data", { path: "/" })
 
       if (data) {
         data = JSON.parse(data)

diff --git a/plugin.rb b/plugin.rb
@@ -1,6 +1,6 @@
 # name: mozilla-iam
 # about: A plugin to integrate Discourse with Mozilla's Identity and Access Management (IAM) system
-# version: 1.4.1
+# version: 1.4.2
 # authors: Leo McArdle
 # url: https://github.com/mozilla/discourse-mozilla-iam