merge: relax websocket rate limits (!601)
View MR for information: https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/601 Approved-by: Julia Johannesen <julia@insertdomain.name> Approved-by: Marie <2-Marie@users.noreply.activitypub.software>
This commit is contained in:
		
						commit
						5ae21970d7
					
				
					 1 changed files with 6 additions and 3 deletions
				
			
		| 
						 | 
				
			
			@ -100,7 +100,6 @@ export class StreamingApiServerService {
 | 
			
		|||
				key: 'wsconnect',
 | 
			
		||||
				duration: ms('5min'),
 | 
			
		||||
				max: 32,
 | 
			
		||||
				minInterval: ms('1sec'),
 | 
			
		||||
			})) {
 | 
			
		||||
				socket.write('HTTP/1.1 429 Rate Limit Exceeded\r\n\r\n');
 | 
			
		||||
				socket.destroy();
 | 
			
		||||
| 
						 | 
				
			
			@ -145,10 +144,14 @@ export class StreamingApiServerService {
 | 
			
		|||
			}
 | 
			
		||||
 | 
			
		||||
			const rateLimiter = () => {
 | 
			
		||||
				// rather high limit, because when catching up at the top of a
 | 
			
		||||
				// timeline, the frontend may render many many notes, each of
 | 
			
		||||
				// which causes a message via `useNoteCapture` to ask for
 | 
			
		||||
				// realtime updates of that note
 | 
			
		||||
				return this.rateLimitThis(user, requestIp, {
 | 
			
		||||
					key: 'wsmessage',
 | 
			
		||||
					duration: ms('5sec'),
 | 
			
		||||
					max: 256,
 | 
			
		||||
					duration: ms('2sec'),
 | 
			
		||||
					max: 4090,
 | 
			
		||||
				});
 | 
			
		||||
			};
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue