mixpanel.reset() is called when users logout in order to prevent profile hijacking and

Yes, I really think this is a case where you should contact <a href="mailto:support@mi

mixpanel.reset() firefox issue,about mixpanel/mixpanel-js

vcarel commented on June 3, 2024 1

Hi,
I'm on Chrome and the reset() method does not delete the mp_xxxx cookie on my side... Is it supposed to work ?

from mixpanel-js.

tdumitrescu commented on June 3, 2024

Thanks for bringing this up. To help us reproduce, is there any way you could let us know:

the full URL you're seeing this behavior on (subdomains are important)
any custom configuration you're passing to mixpanel.init()

?

If any of this info is private or sensitive, just email [email protected] with it.

from mixpanel-js.

vcarel commented on June 3, 2024

It looks like reset takes time... but when I logout I do a page refresh so to ensure I get started with a new state. So it actually works from time to time.

Why not returning a Promise on reset ?

from mixpanel-js.

tdumitrescu commented on June 3, 2024

There's nothing about the reset code that ought to be anything but instantaneous (

mixpanel-js/src/mixpanel-core.js

Lines 1348 to 1356 in f21586c

    
           /** 
        
            * Clears super properties and generates a new random distinct_id for this instance. 
        
            * Useful for clearing data when a user logs out. 
        
            */ 
        
           MixpanelLib.prototype.reset = function() { 
        
               this['persistence'].clear(); 
        
               this._flags.identify_called = false; 
        
               this.register_once({'distinct_id': _.UUID()}, ''); 
        
           };

and

mixpanel-js/src/mixpanel-core.js

Lines 402 to 413 in f21586c

    
           MixpanelPersistence.prototype.remove = function() { 
        
               // remove both domain and subdomain cookies 
        
               this.storage.remove(this.name, false); 
        
               this.storage.remove(this.name, true); 
        
           }; 
        
           // removes the storage entry and deletes all loaded data 
        
           // forced name for tests 
        
           MixpanelPersistence.prototype.clear = function() { 
        
               this.remove(); 
        
               this['props'] = {}; 
        
           };

). This might be more quickly resolved for you by running through your particular setup with our quite knowledgeable and helpful support staff at [email protected].

from mixpanel-js.

vcarel commented on June 3, 2024

So I'm getting a random behaviour. Sometimes it works, most of the time it doesn't. There's no log or anything which can help me to debug. Le mer. 22 févr. 2017 18:31, tdumitrescu <[email protected]> a écrit :

…

There's nothing about the reset code that ought to be anything but instantaneous ( https://github.com/mixpanel/mixpanel-js/blob/f21586cf441b5461919bdba1d6257a2e9c084155/src/mixpanel-core.js#L1348-L1356 and https://github.com/mixpanel/mixpanel-js/blob/f21586cf441b5461919bdba1d6257a2e9c084155/src/mixpanel-core.js#L402-L413). This might be more quickly resolved for you by running through your particular setup with our quite knowledgeable and helpful support staff at ***@***.*** — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#108 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABeD5ekgoGLXY48kAbZu-vaxvLq8IZwnks5rfHDYgaJpZM4Kx9Re> .

from mixpanel-js.

tdumitrescu commented on June 3, 2024

Yes, I really think this is a case where you should contact [email protected] to debug 1-on-1 with someone.

from mixpanel-js.

mixpanel.reset() firefox issue about mixpanel-js HOT 6 OPEN

Comments (6)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	/**
	* Clears super properties and generates a new random distinct_id for this instance.
	* Useful for clearing data when a user logs out.
	*/
	MixpanelLib.prototype.reset = function() {
	this['persistence'].clear();
	this._flags.identify_called = false;
	this.register_once({'distinct_id': _.UUID()}, '');
	};

	MixpanelPersistence.prototype.remove = function() {
	// remove both domain and subdomain cookies
	this.storage.remove(this.name, false);
	this.storage.remove(this.name, true);
	};

	// removes the storage entry and deletes all loaded data
	// forced name for tests
	MixpanelPersistence.prototype.clear = function() {
	this.remove();
	this['props'] = {};
	};