Skip to content

fix: add management command to expire IPS bundles#1810

Merged
staceybeard merged 30 commits intomainfrom
SB.expire-ips-bundles
Mar 9, 2026
Merged

fix: add management command to expire IPS bundles#1810
staceybeard merged 30 commits intomainfrom
SB.expire-ips-bundles

Conversation

@staceybeard
Copy link
Member

Description

Created a new management command to delete expired IPS bundles 1 hour after their creation. This is done by checking the bundles' last modified time.

This delay was chosen as an easy solution for the following SHL constraint: https://docs.smarthealthit.org/smart-health-links/spec/#fileslocation-links

@staceybeard staceybeard self-assigned this Nov 24, 2025
@staceybeard staceybeard changed the title Draft: fix: add management command to expire IPS bundles fix: add management command to expire IPS bundles Feb 26, 2026
Copy link
Member

@mschoettle mschoettle left a comment

Choose a reason for hiding this comment

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

Great work! Added some comments. Let me know if anything is unclear.

@staceybeard staceybeard requested a review from mschoettle March 5, 2026 20:08
Copy link
Member

@mschoettle mschoettle left a comment

Choose a reason for hiding this comment

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

Looks quite good. Leaving some minor comments but approving otherwise.

Co-authored-by: Matthias Schoettle <git@mattsch.com>
@staceybeard staceybeard merged commit 288d0e7 into main Mar 9, 2026
7 checks passed
@staceybeard staceybeard deleted the SB.expire-ips-bundles branch March 9, 2026 19:16
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