Skip to content

Conversation

@danielendler
Copy link
Owner

Motivation

  • Raise minimums for security auditing and cryptography to keep dependency scanners effective and reduce known CVE exposure.
  • Remove a Bandit B112 pattern in the runtime registry to avoid noisy security warnings while preserving fallback behavior when handlers fail.

Description

  • Updated pyproject.toml to bump cryptography to >=46.0.0 and dev tooling minimums: bandit>=1.9.3, safety>=3.7.0, semgrep>=1.149.0, and pip-audit>=2.10.0.
  • Refactored datason/type_registry.py to use local result variables (can_handle, matches, name) instead of try/except/continue, preserving the previous skip-on-error semantics without triggering Bandit B112.
  • Removed the stale rust/Cargo.lock file from the tree.

Testing

  • Ran bandit -c pyproject.toml -r datason/ and confirmed the prior B112 findings were cleared and the second scan reported no issues.
  • Installed bandit==1.9.3 locally to run the scan, and the installation completed successfully.

Codex Task

@danielendler
Copy link
Owner Author

DataSON PR Performance Analysis
PR #90 | Commit: 25a01e9

Performance Check - Improvements Detected

🚀 Performance Improvements

Metric Baseline Current Change
json_safe_simple_serialize_time 63.641μs 60.191μs -5.4%

✅ Stable Metrics (7 metrics within tolerance)


Regression Detection Thresholds:

  • 🚫 Fail: >25% degradation
  • ⚠️ Warn: >10% degradation
  • 📋 Notice: >5% degradation

Generated at 2026-01-29 21:36:02 UTC

🚀 DataSON PR Performance Analysis

PR #90 | Commit: 25a01e9df23cd237a0519d214fd38e685f84b67f

📊 Benchmark Results

Suite: pr_optimized | Tests Run: 5 | Success Rate: 100.0%

🎯 DataSON Performance Summary

Metric Current Result Baseline Change Status
Serialization (avg) 0.276 ms 0.265 ms +4.2%
Deserialization (avg) 0.665 ms 0.663 ms +0.3%
Success Rate 100.0% 100.0% No change
Performance Range 0.060 - 0.508 ms Min to max serialization times N/A

📋 Test Scenarios

Scenario Status Serialization Deserialization
Json Safe Simple ✅ Passed 0.06 ms 0.107 ms
Json Safe Nested ✅ Passed 0.133 ms 0.285 ms
Object Datetime Heavy ✅ Passed 0.508 ms 0.616 ms
Object Api Response ✅ Passed 0.49 ms 0.709 ms
Ml Complex Data ✅ Passed 0.187 ms 1.61 ms

📈 Baseline Comparison

📊 Performance Changes (Below Threshold)

  • Serialization: 4.2% slower
  • Deserialization: 0.3% slower

✅ Status: Ready for Review

All benchmarks passed! No significant performance regressions detected.


Generated by datason-benchmarks • Comprehensive Performance Analysis


Generated by datason-benchmarks • Comprehensive Performance Analysis

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants