Rate limit ws joins (#2171)

* Removing async on mutex lock fn.

* Removing redundant ip

* Don't check rate limit twice.

diff --git a/crates/websocket/src/chat_server.rs b/crates/websocket/src/chat_server.rs
index d9de90dbef8c54d700f6651713cd169a3c850028..4dafafd8e3cf93a1f8f6565a8604f7964ea6f7fb 100644 (file)
--- a/crates/websocket/src/chat_server.rs
+++ b/crates/websocket/src/chat_server.rs
@@ -485,7 +485,7 @@ impl ChatServer {
           UserOperationCrud::CreatePost => rate_limiter.post().check(ip),
           UserOperationCrud::CreateCommunity => rate_limiter.register().check(ip),
           UserOperationCrud::CreateComment => rate_limiter.comment().check(ip),
-          _ => rate_limiter.message().check(ip),
+          _ => true,
         };
         let fut = (message_handler_crud)(context, msg.id, user_operation_crud, data);
         (passed, fut)
@@ -493,7 +493,7 @@ impl ChatServer {
         let user_operation = UserOperation::from_str(op)?;
         let passed = match user_operation {
           UserOperation::GetCaptcha => rate_limiter.post().check(ip),
-          _ => rate_limiter.message().check(ip),
+          _ => true,
         };
         let fut = (message_handler)(context, msg.id, user_operation, data);
         (passed, fut)