chore: 보안 취약 의존성 업데이트 - #585
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
More reviews will be available in 20 minutes and 9 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more credits in the billing tab to continue. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (6)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 93b42bb7e2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "version": "10.5.0", | ||
| "resolved": "https://registry.npmjs.org/glob/-/glob-10.5.0.tgz", | ||
| "integrity": "sha512-DfXN8DfhJ7NH3Oe7cFmu3NCu1wKbkReJ8TorzSAFbSKrlNaQSKfIzqYqVY8zlbs2NLBbWpRiU52GX2PbaBVNkg==", | ||
| "deprecated": "Old versions of glob are not supported, and contain widely publicized security vulnerabilities, which have been fixed in the current version. Please update. Support for old versions may be purchased (at exorbitant rates) by contacting i@izs.me", |
There was a problem hiding this comment.
Avoid locking the CLI to deprecated glob
When packages/bruno-api-typescript is installed with its standalone npm lockfile, this now resolves glob to 10.5.0, and the lockfile itself records that release as deprecated because old glob versions contain public security vulnerabilities fixed in the current version. Since this change is explicitly a security dependency update, leaving a deprecated vulnerable direct dependency here will keep standalone installs/audits of this package failing; upgrade to a non-deprecated glob release or remove the unused dependency.
Useful? React with 👍 / 👎.
관련 이슈
작업 내용
axios,firebase-admin,@sentry/nextjs,vite,vitest,jsdom,turbo,@commitlint/*pnpm.overrides로 patched 버전을 고정했습니다.protobufjs,form-data,minimatch,picomatch,rollup,svgo,ws,yaml등packages/bruno-api-typescript의 별도 npm lockfile도 업데이트하고, npmoverrides를 추가했습니다.특이 사항
pnpm audit와packages/bruno-api-typescript의npm audit는 모두 0건입니다.vitest는 보안 알림 정리를 위해3.x에서4.x로 올렸고, admin 테스트는 통과했습니다.@tailwindcss/vite의 Vite peer range,nitro > unstorage의chokidarpeer. 관련 ci/build는 통과했습니다.리뷰 요구사항 (선택)
pnpm.overrides범위가 과하게 넓지 않은지 확인 부탁드립니다.firebase-admin/@sentry/nextjs/vitest업데이트 영향 범위를 한 번 더 봐주세요.검증
git diff --checkpnpm install --frozen-lockfilepnpm audit --json→ vulnerabilities 0건cd packages/bruno-api-typescript && npm audit --json→ vulnerabilities 0건pnpm --filter @solid-connect/web run ci:checkpnpm --filter @solid-connect/university-web run ci:checkpnpm --filter @solid-connect/admin run ci:checkpnpm --filter @solid-connect/admin testNODE_ENV=production UNIVERSITY_WEB_DOMAIN=https://university-web.ci.local pnpm --filter @solid-connect/web run buildNODE_ENV=production pnpm --filter @solid-connect/university-web run buildNODE_ENV=production pnpm --filter @solid-connect/admin run buildpnpm --filter bruno-api-typescript run buildpnpm --filter @solid-connect/api-schema run typecheckgit pushpre-push parity checks: web/university-web/adminci:check및 production build 통과확인된 기존 테스트 이슈
pnpm --filter bruno-api-typescript test는tests/fixtures/brunofixture 디렉터리가 없어 실패합니다. 이번 dependency 변경 전제와 별개로 fixture가 repo에 없는 상태입니다.