Skip to content
Snippets Groups Projects

relax websocket rate limits

Merged dakkar requested to merge feature/laxer-ws-rate-limits into develop

What does this PR do?

We have just deployed the websocket rate limits to https://dev.joinsharkey.org/ and we can easily hit them with normal usage, so we need to relax them.

  • the frontend opens 2 websockets at startup (I'm not completely clear why), and that minInterval:1sec was breaking the second connection
  • as the comment says, "catching up" generates many noteSubscribe messages

Not really tested, because I'm once again making up numbers. Finding a decent middle ground between "nobody can use the site" and "anyone can crash the site" is not that easy!

Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • These improved limits seem good to me, it works well on localhost.

  • Julia approved this merge request

    approved this merge request

  • Marie resolved all threads

    resolved all threads

  • Marie approved this merge request

    approved this merge request

  • dakkar mentioned in commit 5ae21970

    mentioned in commit 5ae21970

  • merged

  • Please register or sign in to reply
    Loading