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.

Sign up for GitHub

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

Jump to bottom

Use the new rate limit MemoryStore.shutdown #937

Merged
merged 1 commit into from Sep 5, 2022
Merged

Use the new rate limit MemoryStore.shutdown #937

merged 1 commit into from Sep 5, 2022

Conversation

codyebberson
Copy link
Member

Upgrade express-rate-limit to 6.6.0 and use the new MemoryStore.shutdown

express-rate-limit/express-rate-limit#322

Thank you @nfriedly and @gamemaker1!

@codyebberson codyebberson requested a review from a team as a code owner September 5, 2022 16:45
@vercel
Copy link

vercel bot commented Sep 5, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Ignored Deployments
Name Status Preview Updated
medplum-graphiql ⬜️ Ignored (Inspect) Sep 5, 2022 at 4:45PM (UTC)
medplum-storybook ⬜️ Ignored (Inspect) Sep 5, 2022 at 4:45PM (UTC)
medplum-www ⬜️ Ignored (Inspect) Sep 5, 2022 at 4:45PM (UTC)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 94.391% when pulling 82c8c24 on cody-use-rate-limit-memory-store into d59df22 on main.

@sonarcloud
Copy link

sonarcloud bot commented Sep 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codyebberson codyebberson merged commit 93c91cf into main Sep 5, 2022
@codyebberson codyebberson deleted the cody-use-rate-limit-memory-store branch September 5, 2022 18:00
@reshmakh reshmakh added the security Security best practices and enhancements label Sep 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rahul1 rahul1 rahul1 approved these changes

Assignees
No one assigned
Labels
security Security best practices and enhancements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@codyebberson @coveralls @rahul1 @reshmakh