Skip to content
This repository has been archived by the owner on Jun 8, 2018. It is now read-only.

Options button broken in private browsing mode #214

Closed
hfr-hfr opened this issue Oct 19, 2017 · 8 comments
Closed

Options button broken in private browsing mode #214

hfr-hfr opened this issue Oct 19, 2017 · 8 comments

Comments

@hfr-hfr
Copy link

hfr-hfr commented Oct 19, 2017

When in Firefox's "private mode", toolbar widget is clickable and opens its info window, but:

  • "Options" not clickable
  • Doesn't show the emulated CDN, although counter indicates action

x

When in Firefox's "normal mode", the above works as intended.

@Thorin-Oakenpants
Copy link

confirmed in FF56. Info not shown (domain, library, whitelisting button). Options button is clickable but does not do anything. Counter increases by 2 on each test (if that's meant to be correct, it also does this in normal windows)

normal windows
dc-normal

private browsing mode
dc-private

@ghost
Copy link

ghost commented Oct 25, 2017

Browser console error that is thrown: (tested on Nightly)

TypeError: popup.backgroundPage is undefined [Learn More] popup.js:351:1

@Thorin-Oakenpants Wow dude, 119269 injected resources ^^

@Kasijjuf
Copy link

FWIW, Privacy Badger does a similar thing in Private browsing mode. The list of trackers only shows up on normal mode.

@Synzvato
Copy link
Owner

Synzvato commented Oct 26, 2017

Hi everyone, and thanks for sharing your detailed findings and concerns!

As mentioned by @Kasijjuf, any website-specific information is intentionally absent from private browsing mode. This might change in the future. Either way, the options button should not be unresponsive.

I'll dedicate this issue to the underlying bug. Feel free to open up a separate issue, if you are interested in discussing the extension's behavior in private mode. Thanks again! I'll be sure to keep you all posted.

@Synzvato Synzvato changed the title Widget not responsive in Firefox's "private mode" Options button broken in private browsing mode Oct 26, 2017
i4xB0y pushed a commit to i4xB0y/decentraleyes that referenced this issue Oct 31, 2017
@Synzvato Synzvato added this to the v2.0.1 milestone Nov 7, 2017
@Synzvato
Copy link
Owner

Synzvato commented Nov 8, 2017

Counter increases by 2 on each test ([...], it also does this in normal windows)

@Thorin-Oakenpants This is expected behavior. It's because the testing utility requests the very same resource twice. So, two injections occur, but only one unique resource is injected. Good catch though!

@Synzvato
Copy link
Owner

Decentraleyes v2.0.1 has since been published. Installing the update should solve the issue.

@Synzvato
Copy link
Owner

Synzvato commented May 31, 2018

@hfr-hfr @Thorin-Oakenpants I have implemented a messaging system in order to achieve complete feature parity, so v2.0.4 doesn't lack any features when used within Private Browsing contexts.

@Thorin-Oakenpants
Copy link

Thorin-Oakenpants commented May 31, 2018

I saw that 👍 .. still increases the count by two (not that it matters) - nvm, that's the test, not PB mode specific :headbutt:

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

4 participants