Skip to content

Convert from datetime.timedelta to int in objectTTL properties.#1926

Merged
dirkkul merged 1 commit intomainfrom
jose/object-ttl-to-dict
Jan 15, 2026
Merged

Convert from datetime.timedelta to int in objectTTL properties.#1926
dirkkul merged 1 commit intomainfrom
jose/object-ttl-to-dict

Conversation

@jfrancoa
Copy link
Copy Markdown
Contributor

The objectTTL deltaTTL returns a datetime.timedelta object, therefore, when invoking todict() we need to cast into int so that we are returning a well formed json.

The objectTTL deltaTTL returns a datetime.timedelta object,
therefore, when invoking todict() we need to cast into int so
that we are returning a well formed json.
Copy link
Copy Markdown

@orca-security-eu orca-security-eu Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.37%. Comparing base (fd58638) to head (2a1f6c3).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1926      +/-   ##
==========================================
- Coverage   86.45%   86.37%   -0.08%     
==========================================
  Files         274      274              
  Lines       19966    19977      +11     
==========================================
- Hits        17261    17256       -5     
- Misses       2705     2721      +16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jfrancoa jfrancoa requested a review from dirkkul January 15, 2026 11:06
@dirkkul dirkkul merged commit 3bdeca3 into main Jan 15, 2026
119 of 121 checks passed
@dirkkul dirkkul deleted the jose/object-ttl-to-dict branch January 15, 2026 12:21
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.

3 participants