Skip to content

chore(deps): upgraded to latest dependencies#242

Open
Sourav-kashyap wants to merge 1 commit intomasterfrom
ratelimiter
Open

chore(deps): upgraded to latest dependencies#242
Sourav-kashyap wants to merge 1 commit intomasterfrom
ratelimiter

Conversation

@Sourav-kashyap
Copy link
Copy Markdown

Description

Upgraded loopback4-ratelimiter to use the latest versions of rate-limiting dependencies, migrate from older versions to the latest stable releases.

Checklist:

  • Performed a self-review of my own code
  • npm test passes on your machine

Build Passed:

Screenshot 2026-03-20 at 3 35 24 PM

TestCases Passed:

Screenshot 2026-03-20 at 3 35 42 PM

@Sourav-kashyap Sourav-kashyap self-assigned this Mar 20, 2026
@Sourav-kashyap Sourav-kashyap added the dependencies Pull requests that update a dependency file label Mar 20, 2026
@Sourav-kashyap Sourav-kashyap force-pushed the ratelimiter branch 4 times, most recently from f48b3d6 to 6592e7f Compare March 20, 2026 13:19
@yeshamavani
Copy link
Copy Markdown
Contributor

@Sourav-kashyap trivy failing

@Sourav-kashyap Sourav-kashyap force-pushed the ratelimiter branch 2 times, most recently from b00af02 to 3a35863 Compare March 23, 2026 14:02
@Sourav-kashyap Sourav-kashyap force-pushed the ratelimiter branch 4 times, most recently from c35ed52 to 66fea91 Compare April 2, 2026 09:39
upgraded to latest dependencies

GH-0
@sonarqubecloud
Copy link
Copy Markdown

SonarQube reviewer guide

Summary: Upgrade GitHub Actions, dependencies, and support express-rate-limit v8 with modernized code patterns.

Review Focus:

  • GitHub Actions upgraded from v3 to v4; Trivy action updated with pinned hash
  • Major dependency updates: commitlint v17→v20, semantic-release plugins, eslint-config-prettier v9→v10, jsdom v21→v29
  • express-rate-limit v6→v8 requires removing name option, handling cache to prevent store-reuse errors
  • Removed StoreProvider pattern, using null datastore for in-memory stores
  • TypeScript upgraded to 5.5.4; Node.js minimum still 20

Start review at: src/providers/ratelimit-action.provider.ts. This file demonstrates the critical changes needed for express-rate-limit v8 compatibility—specifically the removal of unsupported config options and the new rate limiter caching mechanism to prevent store reuse errors. The same pattern appears in the middleware provider and should be verified for consistency.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
48.4% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants