Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider additional check for file age rather than parsed age #21

Open
astrobokonon opened this issue Sep 20, 2018 · 1 comment
Open
Assignees
Labels
question Further information is requested

Comments

@astrobokonon
Copy link
Member

The current setup (which is the same as the one Peter used) is to parse found directories names' and use that as their age, which has a pitfall:

If the code is turned off for a while, data is created in the meantime, and then the code is turned on after the rangeNew window has elapsed, the data could be silently ignored. I think there's a way to catch that scenario and trigger a retransfer, probably as part as Mandos' mantle of responsibilities when verifying that the machines are synced. But it needs testing and could be a looming bug around shutdown periods.

@astrobokonon astrobokonon added the question Further information is requested label Sep 20, 2018
@astrobokonon astrobokonon self-assigned this Sep 20, 2018
@astrobokonon
Copy link
Member Author

Just realized that Mandos uses the same parsed-name-as-age logic as all the rest, so it seems like a modification date (or creation date if I give up on cross-platform) might be the only way to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant