Closed fengmk2 closed 4 months ago
[!WARNING]
Rate Limit Exceeded
@fengmk2 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 20 minutes and 38 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 307792df75b0141ad4e3deb13537035aeab8c8ab and 40f1a263fe9584c6287ee915f6554c6897ae3588.
The recent updates encompass multiple files, focusing on modernizing the Node.js version matrix, enhancing the encryption and decryption mechanisms using NodeRSA
, and refining import statements to leverage ES module syntax. Additionally, a minor configuration change was made in the release workflow, and dependencies were updated in package.json
. These changes collectively aim to improve security, maintain compatibility with newer Node.js versions, and streamline cryptographic operations.
File | Change Summary |
---|---|
.github/workflows/nodejs.yml |
Updated node-version matrix from [16, 18, 20] to [18, 20, 22] in job configurations. |
.github/workflows/release.yml |
Removed checkTest: false configuration. |
app/common/CryptoUtil.ts |
Refactored encryption/decryption functions to use NodeRSA instead of crypto module. |
package.json |
Removed "revert": ["CVE-2023-46809"] , added "node-rsa": "^1.1.1" and "@types/node-rsa": "^1.1.4" . |
test/common/CryptoUtil.test.ts , test/port/webauth/webauthController.test.ts |
Updated import statements to use ES module syntax and added necessary imports for testing cryptographic functions. |
๐ In the land of code, where changes flow, Node versions rise, old ones go. RSA keys now shine so bright, Encrypting data, day and night. Dependencies fresh, imports refined, A safer, swifter app you'll find. ๐
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
New and removed dependencies detected. Learn more about Socket for GitHub โ๏ธ
Package | New capabilities | Transitives | Size | Publisher |
---|---|---|---|---|
npm/@types/node-rsa@1.1.4 | None | +1 |
1.98 MB | types |
๐ฎ Removed packages: npm/mysql2@3.9.7, npm/npm-package-arg@10.1.0, npm/oss-cnpm@5.0.1, npm/p-map@4.0.0, npm/s3-cnpmcore@1.1.2, npm/semver@7.6.2, npm/ssri@8.0.1, npm/type-fest@2.19.0, npm/typescript@5.2.2, npm/ua-parser-js@1.0.37, npm/validate-npm-package-name@3.0.0
็่ตทๆฅไธ็ฎๅ
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.84%. Comparing base (
a51891d
) to head (442dc40
). Report is 1 commits behind head on master.:exclamation: Current head 442dc40 differs from pull request most recent head 40f1a26
Please upload reports for the commit 40f1a26 to get more accurate results.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
revert https://github.com/cnpm/cnpmcore/pull/650
run test on Node.js 22
Summary by CodeRabbit
New Features
Bug Fixes
Chores
Refactor