Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

data file download race condition #2455

Closed
bridiver opened this issue Jul 12, 2016 · 12 comments
Closed

data file download race condition #2455

bridiver opened this issue Jul 12, 2016 · 12 comments

Comments

@bridiver
Copy link
Collaborator

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
If a new data file is available (either first run or file updated), https everywhere, tracking protection and ad blocking will be disabled until they are loaded, but they appear as "on" in the bravery panel. There are a few ways we can handle this:

  1. Leave as-is, but notify the user that shields are down until the file is downloaded
  2. Delay the loading of any pages until the files are downloaded and notify the user of the download progress

When updates are available we should also start immediately with the old files and swap when the new ones are available.

@bridiver
Copy link
Collaborator Author

if we want to be completely sure they are loaded, we could also apply this to locally read files

@diracdeltas
Copy link
Member

I'm in favor of 1 since 2 will always make us look slow whenever there's an update

@bbondy
Copy link
Member

bbondy commented Jul 12, 2016

Strong preferences for 1 over 2.

@bridiver
Copy link
Collaborator Author

actually there is a 3rd option which is to download the files on install

@bridiver
Copy link
Collaborator Author

cc @bradleyrichter

@BrendanEich
Copy link
Member

Download on install == win/win, is it hard?

@diracdeltas
Copy link
Member

Even if they are downloaded on install, we need to check for updates every ~24 hours so I don't think that solves the problem

@diracdeltas
Copy link
Member

NVM, I forgot that the old file will be active while the update is being downloaded, so that does seem like a win win

@bbondy
Copy link
Member

bbondy commented Jul 12, 2016

Download on install is not possible when you have no running code on install. Only Windows has this.

@bridiver
Copy link
Collaborator Author

There are hooks available on Mac when you copy to and delete from /Applications and on Linux a deb or similar can run code

On Jul 12, 2016, at 3:36 PM, Brian R. Bondy notifications@github.com wrote:

download on install is not possible when you have no running code on install. Only Windows has this.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@bbondy
Copy link
Member

bbondy commented Jul 12, 2016

This is very platform dependent though and moves the implementation to something that locks us into the way we are distributing. Longer term I think we'll end up in the mac store for example. Also we could change installers completely to be only msi based one day. Windows store is another place we'll end up in the future.

@bradleyrichter
Copy link
Contributor

#1 is my pref.

We can show the shields button as disabled and display a "please wait while shields are being updated. "

@luixxiul luixxiul added the bug label Nov 16, 2016
@luixxiul luixxiul added this to the Backlog milestone Jan 2, 2017
@bbondy bbondy removed this from the Backlog milestone Oct 19, 2017
@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@bsclifton bsclifton added the stale label Sep 9, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Sep 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants