Closed ershad-paladin closed 2 weeks ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on base/target branches other than the default branch.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
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?
Issues
4 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Description
Problem
audit log entry is missed when dangling issues are closed by policy executor
Solution
made AuditUtils.postAuditTrail generic and used the same common method to insert audit log when dangling issues are closed
Fixes # (issue if any)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:
Other Information:
List any documentation updates that are needed for the Wiki
Summary by CodeRabbit
New Features
POLICY_DISABLED_MSG
to communicate disabled policy status.Improvements
AuditUtils
to accept map structures instead of specific objects.Bug Fixes
PolicyExecutor
with a new condition based on policy status.Refactor
AuditUtils
andCommonUtils
for accepting map structures.