Skip to content

Conversation

arjavdongaonkar
Copy link
Contributor

@arjavdongaonkar arjavdongaonkar commented May 27, 2025

add AttributedOnPolicyEvaluator for vulnerability age-based policy evaluation

Implements production-ready evaluator with caching, error handling, and comprehensive logging. Supports ISO-8601 period formats with injectable dependencies for testing.

Description

Addressed Issue

Additional Details

Checklist

  • I have read and understand the contributing guidelines
  • This PR fixes a defect, and I have provided tests to verify that the fix is effective
  • This PR implements an enhancement, and I have provided tests to verify that it works as intended
  • This PR introduces changes to the database model, and I have added corresponding update logic
  • This PR introduces new or alters existing behavior, and I have updated the documentation accordingly

@owasp-dt-bot
Copy link

owasp-dt-bot commented May 27, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

…icy evaluation

Implements production-ready evaluator with caching, error handling, and comprehensive logging.
Supports ISO-8601 period formats with injectable dependencies for testing.

Signed-off-by: Arjav <[email protected]>
- Remove injectable Clock/ZoneId dependencies, use system defaults
- Eliminate excessive debug logging and defensive exception handling
- Streamline method implementations and reduce code complexity
- Reduce period cache size from 1000 to 100 entries
- Remove unnecessary vulnerability processing counters
- Maintain all core functionality while improving readability

Signed-off-by: Arjav <[email protected]>
- Remove injectable Clock/ZoneId dependencies, use system defaults
- Reduce period cache size from 1000 to 100 entries
- Tests for AttributedOnPolicyEvaluator.java

Signed-off-by: Arjav <[email protected]>
- Remove injectable Clock/ZoneId dependencies, use system defaults
- Reduce period cache size from 1000 to 100 entries
- Tests for AttributedOnPolicyEvaluator.java

Signed-off-by: Arjav <[email protected]>
@arjavdongaonkar arjavdongaonkar force-pushed the feat/attribute-on-policy branch from daa4527 to 31db808 Compare May 28, 2025 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants