Conversation
Create a dedicated unprivileged system user and switch to it before CMD to prevent RCE from granting full root privileges inside the container. Signed-off-by: Tomer Figenblat <[email protected]> Co-authored-by: Cursor <[email protected]>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 for further information. ✨ 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 |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Test Results66 tests 66 ✅ 1s ⏱️ Results for commit 2f8e195. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #951 +/- ##
=======================================
Coverage 96.02% 96.02%
=======================================
Files 1 1
Lines 327 327
=======================================
Hits 314 314
Misses 13 13 🚀 New features to boost your workflow:
|
The Dockerfile already runs as non-root (USER appuser) since #951. Replace ambiguous "(pending fix)" with a firm "must" rule in ci-cd.mdc and code-reviewer agent. Signed-off-by: Tomer Figenblat <[email protected]> Co-authored-by: Cursor <[email protected]>
* chore: split cursor rules into focused files and add tooling index Replace monolithic project-rules.mdc with focused rule files: - project-overview.mdc (alwaysApply) — tech stack summary - python-conventions.mdc (globs: **/*.py) — coding standards - testing.mdc (globs: app/tests/**) — test conventions - ci-cd.mdc (globs: .github/workflows/**) — CI/CD rules - git-workflow.mdc (intelligent) — git/PR conventions - architecture.mdc (intelligent) — handler patterns - cursor-tooling.mdc (alwaysApply) — agents, commands, skills index Signed-off-by: Tomer Figenblat <[email protected]> Co-authored-by: Cursor <[email protected]> * chore: update non-root Dockerfile rule to reflect current state The Dockerfile already runs as non-root (USER appuser) since #951. Replace ambiguous "(pending fix)" with a firm "must" rule in ci-cd.mdc and code-reviewer agent. Signed-off-by: Tomer Figenblat <[email protected]> Co-authored-by: Cursor <[email protected]> --------- Signed-off-by: Tomer Figenblat <[email protected]> Co-authored-by: Cursor <[email protected]>
Summary
appusersystem user andchownthe workdir to it.USER appuserbeforeEXPOSE/CMDso the application runs unprivileged.Validation
actionlintpasses (no workflow changes).