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

Add Privacy Pass extension #2576

Closed
wants to merge 2 commits into from

Conversation

durch
Copy link

@durch durch commented Jun 3, 2019

This PR addresses brave/brave-browser#1677 by adding Privacy Pass extension. Extension is added as a regular extension installed from the Chrome store, in a fashion identical to the IPFS Companion extension.

After brave/brave-browser#2307 has been implemented this will allow users to have a hassle free experience in all contexts. For now it works in normal and incognito modes, but not in Tor mode.

Submitter Checklist:

Test Plan:

  1. Visit brave://settings/extensions and toggle Privacy Pass switch, a prompt as below will be shown if Privacy Pass extension is not currently installed.

Screenshot 2019-06-03 at 14 50 20

2. Click "Cancel" in the prompt, the Privacy Pass switch should be changed back to off. 3. Toggle it on again, and click "Add extension" in the prompt to install it, Privacy Pass extension should now be installed and enabled. 4. Left click Privacy Pass icon opens current status report, as shown below

Screenshot 2019-06-03 at 14 53 39

5. In brave://settings/extensions, toggle Privacy Pass switch off again and visit brave://extensions, Privacy Pass extension should be shown disabled. 6. Enable Privacy Pass switch toggle in brave://settings/extensions again,Privacy Pass extension should be shown enabled in brave://extensions. 7. Remove Privacy Pass extension in brave://extensions, Privacy Pass extension switch toggle in brave://settings/extensions should be shown off.

Reviewer Checklist:

  • New files have MPL-2.0 license header.
  • Request a security/privacy review as needed.
  • Adequate test coverage exists to prevent regressions
  • Verify test plan is specified in PR before merging to source

After-merge Checklist:

  • The associated issue milestone is set to the smallest version that the
    changes has landed on.
  • All relevant documentation has been updated.

@durch durch changed the title Add Privacy Pass extension - https://github.com/brave/brave-browser/i… Add Privacy Pass extension Jun 3, 2019
@bsclifton bsclifton requested a review from riastradh-brave June 5, 2019 21:46
@durch durch requested a review from bridiver as a code owner July 23, 2019 11:24
@rulatir
Copy link

rulatir commented Aug 8, 2019

Is this PR abandoned?

@durch
Copy link
Author

durch commented Sep 30, 2019

@rulatir I really hope not...

@riastradh-brave
Copy link
Contributor

Our main interest in having PrivacyPass included, rather than as just another extension like any other, is for use in private windows with Tor, to reduce the number of captchas that are presented to Tor users by Cloudflare. However, currently private windows with Tor can't use extensions at all. We are working on changing the architecture of private windows with Tor so that they can use extensions, but it's a lot of work with complicated internal design consequences, and it's not ready yet. See brave/brave-browser#1677 and brave/brave-browser#2307 for more details.

@bsclifton
Copy link
Member

Closing as this is a bit stale (needs rebase) and there are the complications that @riastradh-brave mentioned above (not being able to run extensions in Tor private window)

@bsclifton bsclifton closed this Oct 2, 2019
@durch
Copy link
Author

durch commented Oct 2, 2019

@bsclifton we were just in the process of rebasing, we've done it once already, could we have this repoened, as a placeholder if nothing else.

How do you propose we proceed, is there anything we can do to help with the refactor of Tor windows?

@bsclifton bsclifton reopened this Oct 2, 2019
@tildelowengrimm
Copy link
Contributor

Even once the Tor refactor is complete and extensions are usable in Tor windows, this PR doesn't hit all the UX needs, and there are a couple of technical improvements which would be desirable too.

  • Flipping the switch shouldn't show the extension installation prompt. It should just activate the extension like all the other switches in that section.
  • The Privacy Pass UI needs to actually be integrated, not just shown as an installed extension like this. So it needs to match the design of the rest of the browser, including light/dark mode, &c.
  • @bridiver raised a concern with the use of localstorage to keep the passes — that's not an ideal location, and in particular causes trouble sharing passes between Tor and non-Tor contexts.

@durch
Copy link
Author

durch commented Oct 3, 2019

@tomlowenthal thank you for the feedback. We'll get right to work shipping at least the UX improvements.

@bridiver agree that localstorage is not the best place for keeping the passes. We'll be sure to go over that with the CF team as well, any suggestions as to the better location, at least in the context of Tor vs non-Tor?

@rulatir
Copy link

rulatir commented Oct 6, 2019

Am I correct in assuming that we are finally days away from HAVING™ this, or will some other issue pop up that will block this feature for another half-year?

@riastradh-brave
Copy link
Contributor

I'm hoping we'll be able to take advantage of this soon but there are still some issues remaining with getting extensions running in private windows with Tor.

@NumDeP

This comment has been minimized.

@bsclifton
Copy link
Member

Closing as Privacy pass is installable from the Chrome Web Store. It works great in Tor tabs too

I don't think we want to build this into the browser by default
cc: @rebron @diracdeltas

@bsclifton bsclifton closed this Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants