Skip to content

Commit af6a57e

Browse files
committed
Default the same site option to Lax
1 parent 220a4ec commit af6a57e

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/options.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export default {
1616
batchEvents: false,
1717
cookieExpiration: 365 * 10,
1818
cookieName: 'amplitude_id', // this is a deprecated option
19-
sameSiteCookie: 'None',
19+
sameSiteCookie: 'Lax',
2020
cookieForceUpgrade: false,
2121
deferInitialization: false,
2222
disableCookies: false,

test/amplitude-client.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,10 +93,10 @@ describe('AmplitudeClient', function() {
9393
assert.include(getCookie(cookieName).options, 'Secure');
9494
});
9595

96-
it('should set the SameSite cookie option to None by default', () => {
96+
it('should set the SameSite cookie option to Lax by default', () => {
9797
mockCookie();
9898
amplitude.init(apiKey);
99-
assert.include(getCookie(cookieName).options, 'SameSite=None');
99+
assert.include(getCookie(cookieName).options, 'SameSite=Lax');
100100
});
101101

102102
it('should set the sameSite option on a cookie with the sameSiteCookie Option', () => {

0 commit comments

Comments
 (0)