Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement rate limit for RSA operations. #3957

Merged
merged 8 commits into from
Dec 9, 2024
Merged

Implement rate limit for RSA operations. #3957

merged 8 commits into from
Dec 9, 2024

Conversation

scudette
Copy link
Contributor

@scudette scudette commented Dec 9, 2024

When the server first starts up it needs to populate the RSA cache to
mitigate high CPU load. This PR implements a rate limiter to ensure
that RSA operations are made at a fixed limit. This delays populating
the cache in such a way that the server is not immediately loaded at
start up.

Also bugfixes:

  • Export STACK tables works correctly now
  • GetHunt API removes the CompiledCollectorArgs field to avoid sending
    very large grpc responses.
  • Fixed crash in server_frontend_cert() VQL function when run on the
    client.

snyk-bot and others added 5 commits December 9, 2024 23:35
Snyk has created this PR to upgrade recharts from 2.13.2 to 2.13.3.

See this package in npm:
recharts

See this project in Snyk:
https://app.snyk.io/org/scudette/project/76f4d127-566b-42ef-86f4-bdcbc92b90b4?utm_source=github&utm_medium=referral&page=upgrade-pr
Snyk has created this PR to upgrade qs from 6.13.0 to 6.13.1.

See this package in npm:
qs

See this project in Snyk:
https://app.snyk.io/org/scudette/project/76f4d127-566b-42ef-86f4-bdcbc92b90b4?utm_source=github&utm_medium=referral&page=upgrade-pr
Snyk has created this PR to upgrade dompurify from 3.1.7 to 3.2.0.

See this package in npm:
dompurify

See this project in Snyk:
https://app.snyk.io/org/scudette/project/76f4d127-566b-42ef-86f4-bdcbc92b90b4?utm_source=github&utm_medium=referral&page=upgrade-pr
Snyk has created this PR to upgrade webpack from 5.95.0 to 5.96.1.

See this package in npm:
webpack

See this project in Snyk:
https://app.snyk.io/org/scudette/project/76f4d127-566b-42ef-86f4-bdcbc92b90b4?utm_source=github&utm_medium=referral&page=upgrade-pr
When the server first starts up it needs to populate the RSA cache to
mitigate high CPU load. This PR implements a rate limiter to ensure
that RSA operations are made at a fixed limit. This delays populating
the cache in such a way that the server is not immediately loaded at
start up.

Also bugfixes:

* Export STACK tables works correctly now
* GetHunt API removes the CompiledCollectorArgs field to avoid sending
  very large grpc responses.
* Fixed crash in server_frontend_cert() VQL function when run on the
  client.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ scudette
❌ snyk-bot
You have signed the CLA already but the status is still pending? Let us recheck it.

@scudette scudette merged commit ff467f9 into master Dec 9, 2024
2 of 3 checks passed
@scudette scudette deleted the ratelimit branch December 9, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants