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

Add mtime and storage_mtime to db:convert-filecache-bigint #8412

Merged
merged 1 commit into from
Feb 26, 2018

Conversation

agates
Copy link
Contributor

@agates agates commented Feb 17, 2018

Adding some columns that were left out of the bigint conversion for the filecache.

See #7637

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this makes sense as a temp solution yes.

@rullzer rullzer added enhancement 3. to review Waiting for reviews labels Feb 19, 2018
@rullzer rullzer added this to the Nextcloud 14 milestone Feb 19, 2018
@nickvergessen
Copy link
Member

But really a temp solution. Two things I don't like about this:

  1. There should be no MTime yet which is 2038 (because that means the file is to be edited in 20+years, no one can know now. Should really find out why this happens sometimes.
  2. The column should be migrated to datetime one day 🙊

@agates
Copy link
Contributor Author

agates commented Feb 20, 2018

@nickvergessen The only thing I can think of (and I don't have evidence of it right now, as it only happened once or twice) is when I uploaded a PDF from the Android client.

Looking through my files right now, I actually see several that were modified "in 20 years", all of which precisely show "January 18, 2038 9:14 PM". Many of them have existed since I created my server, so I know I haven't touched them myself recently.

Unfortunately I don't have any steps to reproduce it, but I had to make this change to be able to get Nextcloud working again for me :/. Obviously it's a much larger issue for places where modification time is important -- I just run it for myself.

But the storage type seems to be just symptom of another issue.

@ryhaberecht
Copy link

@agates I initially stumbled upon the issue after trying to upload a PDF from my Linux client. The PDF came straight from my bank (web download) and my local file system also stated an mtime from 2038. It never happened again, though.
This special date in 2038 is nothing more than a full 32 bit array of all 1's interpreted as UNIX epoch timestamp. But I have no idea where this originated from. So no idea where to start debugging :-/

@MorrisJobke
Copy link
Member

@nickvergessen @rullzer Should we backport this to stable13?

@MorrisJobke
Copy link
Member

@nickvergessen @rullzer Should we backport this to stable13?

Ping

@rullzer
Copy link
Member

rullzer commented Mar 6, 2018

I have no prefrence here...

@nickvergessen
Copy link
Member

me neither, mtime dates from 2038+ are invalid in my point of view and just burrow other problems.

@rullzer
Copy link
Member

rullzer commented Mar 7, 2018

So the lets keep it at no for now...

@tflidd
Copy link
Contributor

tflidd commented Mar 29, 2018

What's the recommendation for users facing this issue?
https://help.nextcloud.com/t/migration-from-owncloud-10-0-7-to-nextcloud-12-06-failed/29533

@ryhaberecht
Copy link

ryhaberecht commented Mar 30, 2018

On the client you are uploading from: search for files with a modified-timestamp of >= 2038, change the timestamp to a sane value.
On Unixoids a pipe of find and touch should do the trick. Maybe someone can put together the necessary parameters and put them in the migration documentation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants