-
Notifications
You must be signed in to change notification settings - Fork 1
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
Web page not closing when option is selected #3
Comments
Thanks, I think the issue might be the site is blocking scripts that don't have the same origin. Although you are able to enter the code, I cannot guarantee the browser will be able to execute it, iirc. You might see reports about script origin violation in the developer console, I think. |
Oh, no, sorry for the mystification, the script execution probably has nothing to do with that. Are you sure that the regex matches the URL? You can copy the URL of the given page and test it in the tester on the settings page. |
Yes, I tested, with the regex checker and it was OK.
|
Strange... It worked only once... Since then it only redirects me to any custom page I define but does not close the window... |
Yeah, you're right, it matches for me as well. However, for me it seems to work as expected, the window gets closed. Would you mind telling me what your browser, browser version and extension version are? 🤔
That's actually the behaviour I would expect, yeah. I believe execution of scripts should be terminated when the page changes, to prevent malicious code stealing your cookies or whatever. |
Google Chrome |
And what is the version of ScheduleBlock you are using? |
found it: 1.1.6 |
Hmm, I'm not sure what the issue might be, this exact setup seems to be working on my machine. Are you using any extension that might be interfering? Could you try disabling all the other extensions? Also, could you show me the other rules you are using in case that is the issue? |
I don't find anything suspicious about the rules. I take it it doesn't work even with the other extensions disabled? Also, could you try Facebook instead of |
same... |
Anything of interest in the developer console? Unfortunately I can't do anything about this unless I can replicate it 😕 |
I understand. I will try later on another computer... |
@kolcs Hi, did you manage to replicate it on a different machine? |
@kolcs Hi, do you have any new information for me to look into? (I will eventually close this issue unless that is the case) |
Hi, sorry for no reply. Does this happen to you every time you open the page? Does it happen regardless of how you open the page? I actually found one way to replicate this exact warning message, but it's not something I would personally stumble upon accidentally 😅 |
From what I gather, the gist seems to be that for security reasons Chrome (and Firefox as well) doesn't allow closing of windows by scripts, unless in specific circumstances. For whatever reason for you the circumstances are never met, but it generally affects all users, as I said above. Unfortunately this is completely out of the extension's control, and achieving this functionality would mean circumventing Chrome's security, and then again every time the given vulnerability is fixed. Currently it seems I can only recommend using one of the other actions instead of closing the window, and perhaps chiming in on this request for change: https://issues.chromium.org/issues/40744259 |
I would like to block notifications about adblockplus updates.
I made the following settings:
However when I open the following page:
https://new.adblockplus.org/en/update?an=adblockpluschrome&ap=chrome&apv=120.0.0.0&av=3.21.1&p=chromium&pv=120.0.0.0
the tab keeps opened. I also tested it with facebook.com.
The text was updated successfully, but these errors were encountered: