Skip to content
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

Block Scripts shield sometimes doesn't work #1829

Closed
cndouglas opened this issue Oct 25, 2018 · 17 comments
Closed

Block Scripts shield sometimes doesn't work #1829

cndouglas opened this issue Oct 25, 2018 · 17 comments
Labels
bug feature/shields The overall Shields feature in Brave. needs-investigation A bug not 100% confirmed/fixed OS/macOS priority/P3 The next thing for us to work on. It'll ride the trains. privacy

Comments

@cndouglas
Copy link
Contributor

cndouglas commented Oct 25, 2018

Description

Edit: See my comment below.

Toggling on the Block Scripts shield does not disable JavaScript. This happens even after reloading the page several times after toggling the switch.

Steps to Reproduce

  1. Open a new window and navigate to https://enable-javascript.com/
  2. Click the Brave Shields button and toggle on Block Scripts.

Actual result:

Despite Block Scripts being enabled, JavaScript is still enabled.

The website text shows "Javascript is enabled in your web browser."

Expected result:

JavaScript should be disabled.

The website text should show "Javascript is disabled in your web browser", like Safari does when JavaScript is disabled:

Reproduces how often:

Sometimes.

Brave version (brave://version info)

Brave | 0.57.2 Chromium: 70.0.3538.67 (Official Build) dev(64-bit)
Revision | 9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS | Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? Yes.

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? No.
  • Is the issue reproducible on the latest version of Chrome? No.

Additional Information

@srirambv srirambv added good first issue feature/shields The overall Shields feature in Brave. labels Oct 25, 2018
@srirambv
Copy link
Contributor

Can't reproduce this on Windows

This is on Beta (0.56.6)
javascript

This is on dev (0.57.2)
javascriptdev

@srirambv srirambv added this to the 1.x Backlog milestone Oct 25, 2018
@cndouglas
Copy link
Contributor Author

Weird. After clearing cookies, caches, and content settings, this no longer reproduces for me.

@srirambv
Copy link
Contributor

I just tried without having to clear cache and it worked. may be @kjozwiak could try and check on mac once.

@cndouglas
Copy link
Contributor Author

I think the content/shield settings got in an inconsistent state (maybe because of an upgrade?). Perhaps clearing the content settings fixed it. I'll definitely keep an eye out for this problem to see if it happens again.

@cndouglas cndouglas changed the title Block Scripts shield is completely broken Block Scripts shield sometimes doesn't work Oct 25, 2018
@srirambv
Copy link
Contributor

@liunkae that would be awesome. If you can check this during the next upgrade would be really helpful to see if the content settings getting messed up

@onmyouji
Copy link

onmyouji commented Oct 30, 2018

I think the content/shield settings got in an inconsistent state (maybe because of an upgrade?). Perhaps clearing the content settings fixed it. I'll definitely keep an eye out for this problem to see if it happens again.

I had the same problem on Brave 0.55.20 / Windows 10
You're right, clearing the Content settings fixed it, just tested it myself.

@onmyouji
Copy link

The problem happens again after the latest update to 0.55.22

@tildelowengrimm tildelowengrimm added needs-investigation A bug not 100% confirmed/fixed priority/P3 The next thing for us to work on. It'll ride the trains. bug and removed good first issue labels Oct 31, 2018
@srirambv
Copy link
Contributor

srirambv commented Nov 5, 2018

Could be related to #1991

@cndouglas
Copy link
Contributor Author

Happened to me again, and I don't think there was an update involved this time (I'm on 0.57.4, the latest dev version). After clearing content settings and restarting the browser, the problem was resolved.

@onmyouji
Copy link

onmyouji commented Nov 9, 2018

@liunkae
Did you toggle your shield off/on? I'm able to reproduce the problem by doing that (issue 1991).

@kjozwiak
Copy link
Member

kjozwiak commented Nov 9, 2018

@onmyouji can you still reproduce the issue on 0.56.12 which was just pushed? If you can still reproduce, can you provide some steps that you're using?

@cndouglas
Copy link
Contributor Author

@onmyouji I don't think so, but I'm not sure. Could be the problem in #1991 because a new dev build has not been released yet with the fix.

@onmyouji
Copy link

@kjozwiak
Just updated to 0.56.12, I think the problem has been fixed now. Scripts are still being blocked after disabling/enabling.

@cezaraugusto
Copy link
Contributor

I guess this has been solved, this is the result I have:

screen shot 2018-11-13 at 6 26 46 pm

@tildelowengrimm
Copy link
Contributor

@kjozwiak Can we get a QA test on this, and either close if it's resolved or get STR if not?

@kjozwiak
Copy link
Member

@tomlowenthal once things start cooling off with 0.57.x, @brave/legacy_qa will take a look and see if we can reproduce and provide STR 👍

@rebron rebron modified the milestone: 1.x Backlog Feb 7, 2019
@cndouglas
Copy link
Contributor Author

I have not experienced this in several months, so I assume it's no longer a problem. Closing.

@bbondy bbondy added this to the Closed / Invalid milestone Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/shields The overall Shields feature in Brave. needs-investigation A bug not 100% confirmed/fixed OS/macOS priority/P3 The next thing for us to work on. It'll ride the trains. privacy
Projects
None yet
Development

No branches or pull requests

8 participants