Issue 9987

Title
Fingerprints could be lost on concurrent update
Priority
bug
Status
resolved
Nosy list
ced, reviewbot, roundup-bot
Assigned to
ced
Keywords
review

Created on 2021-01-11.19:03:17 by ced, last changed 1 month ago by roundup-bot.

Messages

New changeset e958ee136904 by Cédric Krier in branch 'default':
Improve fingerprints against concurrent updates
https://hg.tryton.org/tryton-env/rev/e958ee136904
New changeset fd2fc5a0c059 by Cédric Krier in branch 'default':
Improve fingerprints against concurrent updates
https://hg.tryton.org/tryton/rev/fd2fc5a0c059
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-01-11.19:03:17

As the fingerprints are load at __init__ if the file is changed in between by another client, this last changes will be lost.
The best option would be to lock the file but Python standard lib does not have a cross-platform file locking mechanism.
Loading the content just before saving limit this case and it should be enough for our use case.

History
Date User Action Args
2021-03-31 23:56:08roundup-botsetmessages: + msg65961
2021-03-31 23:56:04roundup-botsetmessages: + msg65960
nosy: + roundup-bot
status: testing -> resolved
2021-03-22 23:41:01reviewbotsetmessages: + msg65781
nosy: + reviewbot
2021-03-22 23:28:56cedsetkeyword: + review
reviews: 325041002
status: in-progress -> testing
2021-03-22 23:22:36cedsetassignedto: ced
status: unread -> in-progress
type: behavior
2021-01-11 19:03:17cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)