feat(cookies): Use the same-site attribute to lax (#8626)
CSFR-prevention is already implemented but adding this doesn't hurt. A brief introduction to Same-Site cookies (and the difference between strict and lax) can be found at https://blog.mozilla.org/security/2018/04/24/same-site-cookies-in-firefox-60/ TLDR: We use lax since we want the cookies to be sent when the user navigates safely from an external site.
This commit is contained in:
		
							parent
							
								
									7607f4778b
								
							
						
					
					
						commit
						6f3d934bc1
					
				
					 3 changed files with 4 additions and 1 deletions
				
			
		| 
						 | 
				
			
			@ -10,6 +10,7 @@ Warden::Manager.after_set_user except: :fetch do |user, warden|
 | 
			
		|||
    expires: 1.year.from_now,
 | 
			
		||||
    httponly: true,
 | 
			
		||||
    secure: (Rails.env.production? || ENV['LOCAL_HTTPS'] == 'true'),
 | 
			
		||||
    same_site: :lax,
 | 
			
		||||
  }
 | 
			
		||||
end
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -20,6 +21,7 @@ Warden::Manager.after_fetch do |user, warden|
 | 
			
		|||
      expires: 1.year.from_now,
 | 
			
		||||
      httponly: true,
 | 
			
		||||
      secure: (Rails.env.production? || ENV['LOCAL_HTTPS'] == 'true'),
 | 
			
		||||
      same_site: :lax,
 | 
			
		||||
    }
 | 
			
		||||
  else
 | 
			
		||||
    warden.logout
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -1,3 +1,3 @@
 | 
			
		|||
# Be sure to restart your server when you modify this file.
 | 
			
		||||
 | 
			
		||||
Rails.application.config.session_store :cookie_store, key: '_mastodon_session', secure: (Rails.env.production? || ENV['LOCAL_HTTPS'] == 'true')
 | 
			
		||||
Rails.application.config.session_store :cookie_store, key: '_mastodon_session', secure: (Rails.env.production? || ENV['LOCAL_HTTPS'] == 'true'), same_site: :lax
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -29,6 +29,7 @@ Devise::Test::ControllerHelpers.module_eval do
 | 
			
		|||
      value: resource.activate_session(warden.request),
 | 
			
		||||
      expires: 1.year.from_now,
 | 
			
		||||
      httponly: true,
 | 
			
		||||
      same_site: :lax,
 | 
			
		||||
    }
 | 
			
		||||
  end
 | 
			
		||||
end
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		
		Reference in a new issue