Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Add Option UI to Enable MCB Rulesets Globally #15070

Merged
merged 2 commits into from
Apr 6, 2018
Merged

Add Option UI to Enable MCB Rulesets Globally #15070

merged 2 commits into from
Apr 6, 2018

Conversation

cschanaj
Copy link
Collaborator

@cschanaj cschanaj commented Apr 4, 2018

This will allow advanced users to enable mixedcontent rulesets easily and globally, it should be particularly useful in Block all unencrypted requests mode where MCB is expected and resolved in #14600

Basic testings were performed by visiting https://www.wwf.or.jp/ (which has a mixedcontent rules)

  • With the option unchecked: WWF.or.jp shown up in Experimental rules and unchecked
  • With the optoin checked: WWF.or.jp shown up in Stable rules and checked

Related: /issues/9971, https://trac.torproject.org/projects/tor/ticket/21323

Re-open #15069

@cschanaj cschanaj closed this Apr 4, 2018
@cschanaj cschanaj reopened this Apr 4, 2018
@cschanaj cschanaj changed the title Add UI to enable MCB rules globally Add UI to Enable MCB Rulesets Globally Apr 4, 2018
@cschanaj cschanaj changed the title Add UI to Enable MCB Rulesets Globally Add Option UI to Enable MCB Rulesets Globally Apr 4, 2018
@cschanaj cschanaj closed this Apr 5, 2018
@cschanaj cschanaj reopened this Apr 5, 2018
@cschanaj
Copy link
Collaborator Author

cschanaj commented Apr 5, 2018

ping @Hainish for reviews and comments.

Copy link
Member

@Hainish Hainish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise looks good!

<!ENTITY https-everywhere.options.importSettings "Import Settings">
<!ENTITY https-everywhere.options.import "Import">
<!ENTITY https-everywhere.options.showCounter "Show Counter">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was initially in the menu for the old XPCOM extension, which is why it is under the "menu" section. However, changing it to reflect the latest location will clear existing translations, since it is will be tagged by another variable name in transifex. I'd keep it how it was.

@cschanaj
Copy link
Collaborator Author

cschanaj commented Apr 6, 2018

@Hainish done. thanks!!

@Hainish Hainish merged commit 3633095 into EFForg:master Apr 6, 2018
@cschanaj cschanaj deleted the enable-mixed-rulesets-option-ui branch April 7, 2018 00:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants