vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix some problems with off by one in the id condition when deleteing temporary access tokens.

Browse source
This commit is contained in:
Jake Moshenko 2015-02-20 16:23:36 -05:00
parent f7b5221391
commit 24ab0ae53a
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
util/delete_access_tokens.py
View file

@ -8,10 +8,10 @@ from app import app
logger = logging.getLogger(__name__) logger = logging.getLogger(__name__)
BATCH_SIZE = 5000 BATCH_SIZE = 1000
def delete_temporary_access_tokens(older_than): def delete_temporary_access_tokens(older_than):
# Find the higest ID up to which we should delete # Find the highest ID up to which we should delete
up_to_id = (AccessToken up_to_id = (AccessToken
.select(AccessToken.id) .select(AccessToken.id)
.where(AccessToken.created < older_than) .where(AccessToken.created < older_than)
@ -29,9 +29,9 @@ def delete_temporary_access_tokens(older_than):
start_time = datetime.utcnow() start_time = datetime.utcnow()
(AccessToken (AccessToken
.delete() .delete()
.where(AccessToken.id > starting_at_id, .where(AccessToken.id >= starting_at_id,
AccessToken.id < up_to_id, AccessToken.id < up_to_id,
AccessToken.temporary == 1, AccessToken.temporary == True,
~(AccessToken.id << access_tokens_in_builds)) ~(AccessToken.id << access_tokens_in_builds))
.execute()) .execute())