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

Pin serde due to security concerns #20

Merged
merged 1 commit into from
Aug 19, 2023
Merged

Pin serde due to security concerns #20

merged 1 commit into from
Aug 19, 2023

Conversation

Jonxslays
Copy link
Owner

Summary

This PR pins serde to v1.0.171 as they recently started shipping precompiled binaries for derive.

Checklist

  • I have run cargo test and all tests pass.
  • I have run cargo fmt and the code is formatted.
  • I have run cargo clippy in pedantic mode and refactored.
  • I have included documentation for any new structs or methods.
  • I have updated tests for any code I addded/changed/deleted.
  • I have updated the CHANGELOG to include my changes.

Related Issues

@Jonxslays Jonxslays merged commit 00eaafd into master Aug 19, 2023
12 checks passed
@Jonxslays Jonxslays deleted the task/serde-derive branch August 19, 2023 17:26
Jonxslays pushed a commit that referenced this pull request Aug 27, 2023
* feat: ratelimiting for keys

* refactor(router.ts): rename ratelimit tokens to refillRate and duration to refillInterval for clarity
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.

1 participant