Fixes #13628: replaced invalid security email.#13787
Conversation
|
Warning Rate limit exceeded@Pa-Touche has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 26 minutes and 32 seconds before requesting another review. ⌛ 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. 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughCorrected the security reporting email in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In @docs/SECURITY.md:
- Around line 7-9: The nested Markdown list items currently indented with 4
spaces should use 2-space indentation to satisfy lint rule MD007; update the
three lines ("Reporting a Security Bug", "Disclosure Policy", "Comments on this
Policy") to be indented by 2 spaces instead of 4 so the list nesting level is
correct and lint warnings are resolved.
- Line 22: Update the SECURITY.md entry that currently says "Report security
bugs by emailing at **info@sormas.org**." to use a dedicated, monitored security
address (e.g., replace the string "info@sormas.org" with "security@sormas.org"
or your org's official security contact), and ensure the documentation notes
that this mailbox is actively monitored and routed to the security team for
timely handling.
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/SECURITY.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
docs/SECURITY.md
7-7: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
8-8: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
9-9: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
22-22: Bare URL used
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: SORMAS CI
🔇 Additional comments (1)
docs/SECURITY.md (1)
24-27: Good improvement to set clear expectations.The expanded acknowledgment and follow-up process provides better transparency for security reporters about response timelines and next steps.
|
SonarCloud analysis: https://sonarcloud.io/dashboard?id=SORMAS-Project&pullRequest=13787 |
security@sormas.org -> info@sormas.org
9641602 to
dc32fc2
Compare
|
SonarCloud analysis: https://sonarcloud.io/dashboard?id=SORMAS-Project&pullRequest=13787 |
3 similar comments
|
SonarCloud analysis: https://sonarcloud.io/dashboard?id=SORMAS-Project&pullRequest=13787 |
|
SonarCloud analysis: https://sonarcloud.io/dashboard?id=SORMAS-Project&pullRequest=13787 |
|
SonarCloud analysis: https://sonarcloud.io/dashboard?id=SORMAS-Project&pullRequest=13787 |
security@sormas.org -> info@sormas.org
Fixes #13628
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.