search

Meteor-Community-Packages / meteor-timesync

NTP-style time synchronization between server and client, and facilities to use server time reactively in Meteor applications.
https://packosphere.com/mizzao/timesync
MIT License
118 stars 36 forks source link

Quota Exceeded Error #48

Closed houshuang closed 7 years ago

houshuang commented 7 years ago

I am testing an app with a few hundred users, using Timesync, and a minority of users (on iOS 10, Safari) get Quota Exceeded, Error Initializing Storage for FS.HTTP. I think there should be a fall-back for this to avoid crashing the entire app - either just storing it in a variable, rather than using local storage or what it's doing, or just not syncing the time at all, but at least not crashing the app.

mizzao commented 7 years ago

This package doesn't use LocalStorage so I think something else in your app is causing the error.

houshuang commented 7 years ago

So this is not from your package?

screen shot 2017-10-02 at 17 09 20
mizzao commented 7 years ago

Nope. My best guess is CollectionFS:

https://github.com/CollectionFS/Meteor-CollectionFS/blob/b9af68a1e28e348a5ca97e52311bca3941c07aff/packages/access-point/access-point-client.js#L41

You should tell them to use this package instead :)

houshuang commented 7 years ago

Haha, that's funny. I apologize - I just saw timeDiff, and I automatically assumed that it was your package. Apologize. I am planning ot remove CFS and go with an external file server (minio) anyway, I'm worried about the performance of CFS, and it doesn't seem very up-to-date. Another reason to remove that code.

Thanks again!

On Mon, Oct 2, 2017 at 5:18 PM, Andrew Mao notifications@github.com wrote:

Nope. My best guess is CollectionFS:

https://github.com/CollectionFS/Meteor-CollectionFS/blob/ b9af68a1e28e348a5ca97e52311bca3941c07aff/packages/access- point/access-point-client.js#L41

You should tell them to use this package instead :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mizzao/meteor-timesync/issues/48#issuecomment-333566667, or mute the thread https://github.com/notifications/unsubscribe-auth/AADwh6F7nkcS_iBvJ-FcgHaKwh3MyCxMks5soP7QgaJpZM4Pqv7z .

-- http://reganmian.net/blog -- Random Stuff that Matters