Skip to content

Commit 76fded2

Browse files
committed
still send referrer if sessionStorage disabled
1 parent 25ef740 commit 76fded2

File tree

3 files changed

+24
-16
lines changed

3 files changed

+24
-16
lines changed

amplitude.js

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -496,16 +496,20 @@ Amplitude.prototype._saveReferrer = function(referrer) {
496496
identify.setOnce('initial_referring_domain', referring_domain);
497497

498498
// only save referrer if not already in session storage or if storage disabled
499+
var hasSessionStorage = false;
499500
try {
500-
var hasSessionStorage = window.sessionStorage ? true : false;
501-
if ((hasSessionStorage && !window.sessionStorage.getItem(LocalStorageKeys.REFERRER)) || !hasSessionStorage) {
502-
identify.set('referrer', referrer).set('referring_domain', referring_domain);
503-
if (hasSessionStorage) {
504-
window.sessionStorage.setItem(LocalStorageKeys.REFERRER, referrer);
505-
}
501+
if (window.sessionStorage) {
502+
hasSessionStorage = true;
506503
}
507504
} catch (e) {
508-
//log(e); // sessionStorage disabled
505+
// log(e); // sessionStorage disabled
506+
}
507+
508+
if (!hasSessionStorage || (hasSessionStorage && !window.sessionStorage.getItem(LocalStorageKeys.REFERRER))) {
509+
identify.set('referrer', referrer).set('referring_domain', referring_domain);
510+
if (hasSessionStorage) {
511+
window.sessionStorage.setItem(LocalStorageKeys.REFERRER, referrer);
512+
}
509513
}
510514

511515
this.identify(identify);

0 commit comments

Comments
 (0)