URLBlacklist policy not working for startup URLs |
|||||||
Issue descriptionThis seems like a timing issue where the startup URL is successfully loaded before the policies have taken effect. Repro steps (100% repro, only tried Linux), see screencast too: 1) Set a policy for URLBlacklist, for example https://www.yahoo.com 2) Go to chrome://settings and add that address as a startup URL. 3) Close the browser and re-open. Expected: The blocked URL is not loaded. Actual: The blocked URL is loaded. Note that refreshing the page, results in the "blocked" error message finally showing up.
,
Mar 14 2018
I'm able to repo this on Windows so I think it's at least an issue on desktop (haven't tried ChromeOS yet) Assign to Georges as this is an old policy and the original owner has left Chrome already.
,
Mar 14 2018
FYI, I tried this on 64.0.3274.0 (Linux) and it still reproduced, so not a recent regression (perhaps it has always been that way?).
,
Mar 14 2018
dpapad@, could you pls try to repro this on M64 last stable version 64.0.3282.186?
,
Mar 14 2018
,
Mar 14 2018
I am not aware of an easy way to get latest M64 locally, but also don't think is necessary. This is not working on some M64 version, or M65, or ToT (67.0.3371.0). If it had been fixed and merged into any of previous releases, should be already working on ToT.
,
Mar 14 2018
Not considering this as M65 stable blocker per comments #3, #6 and per internal group chat. Thank you.
,
Mar 14 2018
georgesak, my theory is this: the policy code uses a navigation throttle to enforce what is blocked. the first time this navigation throttle is called, it accesses a keyed service which holds the actual block list data structure. it takes time to parse this block list. the throttle doesn't wait very long before deciding that there isn't anything in the block list, and then allows the url to load One approach to fix this is to have the PolicyBlacklistService know that url_blacklist_manager_ hasn't started and return DEFER in PolicyBlacklistNavigationThrottle::WillStartRequest until the url_blacklist_manager_ is ready. When it is ready, you can call the appropriate method on the navigation throttle to continue that request. Happy to discuss.
,
Mar 22 2018
,
May 2 2018
Issue 838943 has been merged into this issue.
,
Oct 31
,
Dec 10
I will take a stab at this one.
,
Dec 10
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by dpa...@chromium.org
, Mar 14 20181.5 MB
1.5 MB View Download