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

Updating third-party filters is horrendously slow on Edge for build 15007 #54

Closed
vemacs opened this issue Jan 16, 2017 · 18 comments
Closed

Comments

@vemacs
Copy link

vemacs commented Jan 16, 2017

Read first: https://github.com/gorhill/uBlock/blob/master/CONTRIBUTING.md

Describe the issue

The progressbar when updating third-party filters hangs about halfway, around the first pull from the web when trying to update filter lists in Edge on Windows 10 15007.

One or more specific URLs where the issue occurs

ms-browser-extension://EdgeExtension_37833NikRollsuBlockOrigin_f8jsg5mm64m62/dashboard.html

Screenshot in which the issue can be seen

Not sure if this is a visual issue, but:

Steps for anyone to reproduce the issue

1. Open settings
2. Click third-party filter lists
3. Click update
4. Wait

Your settings

All default settings

  • Browser/version: Microsoft Edge 15007
  • uBlock Origin version: 1.10.2
Your filter lists

Default filter lists + Anti-Adblock Killer

Your custom filters (if any)
@gorhill
Copy link

gorhill commented Jan 16, 2017

hangs about halfway

There is a timeout of 30 seconds if a server does not respond to a request when fetching a filter list. Did you have to wait more than 30 seconds?

@vemacs
Copy link
Author

vemacs commented Jan 16, 2017

I waited about 5 minutes.

@japborst
Copy link

This used to happen pre- #21. Shouldn't happen on 1.10.2, but perhaps the latest fast build introduces some new issues or is buggy with its communication. My guess it's the same type of issue. I'm still on the slow ring (14986), so unfortunately cannot test this... When refreshing, do the lists still show that an update is available?

@vemacs
Copy link
Author

vemacs commented Jan 16, 2017

Yes, the orange update available indicator does not go away. They do appear to work (although possibly with an older version).

@nicole-ashley
Copy link
Owner

Yes, I've had this happen to me once (out of many times) and mentioned by someone else. It's a tricky one to reproduce.

@vemacs
Copy link
Author

vemacs commented Jan 16, 2017

This seems to happen 100% of the time on 15007 on my machine

@kevin12314
Copy link

I find that in build 15007, "Enabled XMLHttpRequests over Fetch" is by default in about:flags.
With this enabled , I will get error to update user script or import user script with URL in Tempmonkey , and update third-party filters in ublock Origin get vary slow .

@nicole-ashley
Copy link
Owner

Thanks @kevin12314, that gives me somewhere to look. I know the Edge team are doing a lot of work in that area and I'll let them know.

@troskop
Copy link

troskop commented Feb 20, 2017

Tested this on the most-recent Windows10 build (15025).
I cannot confirm this bug. As I tested it, it updated all the lists in less than 2 minutes.

@vemacs
Copy link
Author

vemacs commented Feb 20, 2017

  1. 15025 is not the latest windows 10 build
  2. 2 minutes is still an order of magnitude longer than what ublock takes on other browsers

@troskop
Copy link

troskop commented Feb 20, 2017

@vemacs : Build 15025 is the most-recent stable build. Of course, there is the newer build 15031, but it is marked as unstable/fast-ring.
And as Microsoft recommends, devs should focus on stable versions.

I tested the extension for Edge and the extension for Chrome.
Edge needed 00:52 seconds to update all filters, Chrome needed 00:36 seconds.

@gorhill
Copy link

gorhill commented Feb 20, 2017

Edge needed 00:52 seconds to update all filters, Chrome needed 00:36 seconds

For Chrome, was this uBO 1.11.0? If so, that part of the code has been re-factored and I have shortened the delay between each fetch, from 5 seconds to 2 seconds. Also, with 1.11.0, it's possible to configure the delay between each fetch. The reason for the delay is to be gentle on remote servers, and to be slow-connection friendly.

@vemacs
Copy link
Author

vemacs commented Feb 20, 2017

You're right that this bug can be closed then, as the updating process doesn't outright hang anymore.

@vemacs vemacs closed this as completed Feb 20, 2017
@troskop
Copy link

troskop commented Feb 20, 2017

image
image

@gorhill
These are the versions on my machine.
Both are running with default configs, no changes applied.

And both have been installed recently and have been updated!
These are the newest versions the Windows Store and Chrome were able to install.

@vemacs
Copy link
Author

vemacs commented Mar 11, 2017

This is actually happening again with the latest store build and Win10 15055. The spinners when clicking "Update Filter List" only stop spinning after about 15 minutes for a not-very-large filter selection.

@vemacs vemacs reopened this Mar 11, 2017
@gorhill
Copy link

gorhill commented Mar 11, 2017

I looked into the code to be sure, and the only way I can explain what you observe is if the "Update now" button did not actually send the order to update the filters to the main process. This would mean that the filter lists were actually updated by the auto-updater, which causes filter lists to be slowly updated in the background (auto-updater will update a filter list if needed at most once every two minutes).

@nikrolls That would be the first thing to look at, whether it can happen the "Update now" button does not result in this handler to be called.

@nicole-ashley
Copy link
Owner

@vemacs, are you still experiencing this issue with the latest Windows and uBlock Origin releases?

@nicole-ashley
Copy link
Owner

Closing this due to a lack of activity and being unable to reproduce the issue any longer.

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

6 participants