Skip to content
This repository was archived by the owner on Nov 21, 2025. It is now read-only.

Fix #239 (Google Drive crash on sync) - #240

Open
OMGtechy wants to merge 1 commit into
hwittenborn:developfrom
OMGtechy:develop
Open

Fix #239 (Google Drive crash on sync)#240
OMGtechy wants to merge 1 commit into
hwittenborn:developfrom
OMGtechy:develop

Conversation

@OMGtechy

@OMGtechy OMGtechy commented Nov 29, 2024

Copy link
Copy Markdown

The issue occurs when the remote timestamp in the DB is more recent than the next (i.e. time has gone backwards).

It's not possible to know which is "really" more recent here, so the safest thing to do is to report an error and skip that file.

I am guessing that Google changed how timestamps worked in the 11th November update 🤷‍♀️

The issue occurs when the remote timestamp in the DB is more recent than the next (i.e. time has gone backwards).

It's not possible to know which is "really" more recent here, so the safest thing to do is to report an error and skip that file.
@OMGtechy

Copy link
Copy Markdown
Author

image

Screenshot with the fix running.

@OMGtechy

Copy link
Copy Markdown
Author

Extra info:

The directory issue seems odd (this is a file, not a directory).
The local_path_string is set to /home/username/Google//AsciiArt.txt

I wonder if that double / is causing issues...

@OMGtechy

Copy link
Copy Markdown
Author

Note: the issue still comes up if I rerun; every time I launch the app. But this is definitely better than crashing at least.

@OMGtechy

Copy link
Copy Markdown
Author

Looking at my Google drive in the Web UI, it has the same file listed twice, with different contents. Madness. Removing one of the duplicates seems to also fix the problem more permanently (with or without this PR)

@R0rit0

R0rit0 commented Mar 14, 2025

Copy link
Copy Markdown

I know this is old now, but tested the fix, and it works for me too :)
I guess 2 way sync is not possible because of the timestamp issue, as it generates errors, but it's great to start

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants