diff --git a/composer.lock b/composer.lock index 1498255cea..b7308aec21 100644 --- a/composer.lock +++ b/composer.lock @@ -694,16 +694,16 @@ }, { "name": "phpseclib/phpseclib", - "version": "3.0.51", + "version": "3.0.52", "source": { "type": "git", "url": "https://github.com/phpseclib/phpseclib.git", - "reference": "d59c94077f9c9915abb51ddb52ce85188ece1748" + "reference": "2adaefc83df2ec548558307690f376dd7d4f4fce" }, "dist": { "type": "zip", - "url": "https://api.github.com/repos/phpseclib/phpseclib/zipball/d59c94077f9c9915abb51ddb52ce85188ece1748", - "reference": "d59c94077f9c9915abb51ddb52ce85188ece1748", + "url": "https://api.github.com/repos/phpseclib/phpseclib/zipball/2adaefc83df2ec548558307690f376dd7d4f4fce", + "reference": "2adaefc83df2ec548558307690f376dd7d4f4fce", "shasum": "" }, "require": { @@ -784,7 +784,7 @@ ], "support": { "issues": "https://github.com/phpseclib/phpseclib/issues", - "source": "https://github.com/phpseclib/phpseclib/tree/3.0.51" + "source": "https://github.com/phpseclib/phpseclib/tree/3.0.52" }, "funding": [ { @@ -800,7 +800,7 @@ "type": "tidelift" } ], - "time": "2026-04-10T01:33:53+00:00" + "time": "2026-04-27T07:02:15+00:00" }, { "name": "psr/event-dispatcher", diff --git a/lib/Backgroundjobs/Cleanup.php b/lib/Backgroundjobs/Cleanup.php index e1a5ece068..79ced29322 100644 --- a/lib/Backgroundjobs/Cleanup.php +++ b/lib/Backgroundjobs/Cleanup.php @@ -6,17 +6,17 @@ namespace OCA\Richdocuments\Backgroundjobs; -use OCA\Richdocuments\Db\WopiMapper; use OCP\AppFramework\Utility\ITimeFactory; use OCP\BackgroundJob\TimedJob; use OCP\DB\QueryBuilder\IQueryBuilder; use OCP\IDBConnection; class Cleanup extends TimedJob { + private const EXPIRY_GRACE_PERIOD_SECONDS = 60; + public function __construct( ITimeFactory $time, private IDBConnection $db, - private WopiMapper $wopiMapper, ) { parent::__construct($time); @@ -28,7 +28,7 @@ protected function run($argument) { // Expire template mappings for file creation $query = $this->db->getQueryBuilder(); $query->delete('richdocuments_template') - ->where($query->expr()->lte('timestamp', $query->createNamedParameter(time() - 60, IQueryBuilder::PARAM_INT))); + ->where($query->expr()->lte('timestamp', $query->createNamedParameter(time() - self::EXPIRY_GRACE_PERIOD_SECONDS, IQueryBuilder::PARAM_INT))); $query->executeStatement(); // Expired WOPI access tokens @@ -36,10 +36,9 @@ protected function run($argument) { } private function cleanUpWopiTokens() { - $tokenIds = $this->wopiMapper->getExpiredTokenIds(1000); $query = $this->db->getQueryBuilder(); $query->delete('richdocuments_wopi') - ->where($query->expr()->in('id', $query->createNamedParameter($tokenIds, IQueryBuilder::PARAM_INT_ARRAY))); + ->where($query->expr()->lt('expiry', $query->createNamedParameter(time() - self::EXPIRY_GRACE_PERIOD_SECONDS, IQueryBuilder::PARAM_INT))); $query->executeStatement(); } }