ip_cleanup_scheduler: Make IP and session retention configurable (#18757)
This commit is contained in:
		
							parent
							
								
									c9d6571da8
								
							
						
					
					
						commit
						25e076505e
					
				
					 2 changed files with 12 additions and 2 deletions
				
			
		|  | @ -67,3 +67,11 @@ S3_BUCKET=files.example.com | |||
| AWS_ACCESS_KEY_ID= | ||||
| AWS_SECRET_ACCESS_KEY= | ||||
| S3_ALIAS_HOST=files.example.com | ||||
| 
 | ||||
| # IP and session retention | ||||
| # ----------------------- | ||||
| # Make sure to modify the scheduling of ip_cleanup_scheduler in config/sidekiq.yml | ||||
| # to be less than daily if you lower IP_RETENTION_PERIOD below two days (172800). | ||||
| # ----------------------- | ||||
| IP_RETENTION_PERIOD=31556952 | ||||
| SESSION_RETENTION_PERIOD=31556952 | ||||
|  |  | |||
|  | @ -3,7 +3,8 @@ | |||
| class Scheduler::IpCleanupScheduler | ||||
|   include Sidekiq::Worker | ||||
| 
 | ||||
|   IP_RETENTION_PERIOD = 1.year.freeze | ||||
|   IP_RETENTION_PERIOD = ENV.fetch('IP_RETENTION_PERIOD', 1.year).to_i.seconds.freeze | ||||
|   SESSION_RETENTION_PERIOD = ENV.fetch('SESSION_RETENTION_PERIOD', 1.year).to_i.seconds.freeze | ||||
| 
 | ||||
|   sidekiq_options retry: 0 | ||||
| 
 | ||||
|  | @ -15,7 +16,8 @@ class Scheduler::IpCleanupScheduler | |||
|   private | ||||
| 
 | ||||
|   def clean_ip_columns! | ||||
|     SessionActivation.where('updated_at < ?', IP_RETENTION_PERIOD.ago).in_batches.destroy_all | ||||
|     SessionActivation.where('updated_at < ?', SESSION_RETENTION_PERIOD.ago).in_batches.destroy_all | ||||
|     SessionActivation.where('updated_at < ?', IP_RETENTION_PERIOD.ago).in_batches.update_all(ip: nil) | ||||
|     User.where('current_sign_in_at < ?', IP_RETENTION_PERIOD.ago).in_batches.update_all(sign_up_ip: nil) | ||||
|     LoginActivity.where('created_at < ?', IP_RETENTION_PERIOD.ago).in_batches.destroy_all | ||||
|     Doorkeeper::AccessToken.where('last_used_at < ?', IP_RETENTION_PERIOD.ago).in_batches.update_all(last_used_ip: nil) | ||||
|  |  | |||
		Reference in a new issue