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

re-enable autoconsent on Windows using the "enableIfMainWorldIsSuppoted" setting #1233

Merged
merged 3 commits into from
Aug 22, 2023

Conversation

szanto90balazs
Copy link
Contributor

@szanto90balazs szanto90balazs commented Aug 22, 2023

Asana Task/Github Issue: https://app.asana.com/0/0/1205319036091446/f

cc: @muodov @SlayterDev

Description

In #1212 we temporarily disabled autoconsent.

Now we would like to enable it again, but:

  • old clients (< 0.50) should still have autoconsent disabled
  • new clients (>= 0.50) should have the feature immediately enabled

We wanted to use minSupportedVersion to achieve our goals, but the native service handler don't take minSupportedVersion into account at all and only listen to the global state enabled/disabled flag.

As a workaround and/or improvement:

  • added minSupportedVersion and implemented the proper client side handling of that property
  • added a new setting called enableIfMainWorldIsSuppoted which will temporarily act as a enabled/disabled flag and we'll keep top level global state flag disabled, until almost everyone is updated to v0.50 or higher. Then we will remove the enableIfMainWorldIsSuppoted and flip the global disabled to enabled.

Reference

@github-actions
Copy link

Generated file outputs:

legacy
fingerprinting.json (3 more)
  • fingerprinting.json
  • protections.json
  • trackers-unprotected-temporary.txt
  • trackers-whitelist-temporary.txt

⚠️ File is identical

v1/android-config.json (1 more)
  • v1/android-config.json
  • v2/android-config.json
--- v2/android-config.json
+++ v2/android-config.json
@@ -7069,16 +7069,8 @@
             },
             "exceptions": [],
             "state": "disabled"
         },
-        "voiceSearch": {
-            "exceptions": [],
-            "state": "enabled",
-            "settings": {
-                "minVersion": 33,
-                "excludedManufacturers": []
-            }
-        },
         "webCompat": {
             "exceptions": [],
             "state": "enabled",
             "settings": {
v1/extension-brave-config.json (23 more)
  • v1/extension-brave-config.json
  • v1/extension-bravemv3-config.json
  • v1/extension-chrome-config.json
  • v1/extension-chromemv3-config.json
  • v1/extension-config.json
  • v1/extension-edg-config.json
  • v1/extension-edge-config.json
  • v1/extension-edgmv3-config.json
  • v1/extension-firefox-config.json
  • v1/extension-safarimv3-config.json
  • v1/ios-config.json
  • v1/macos-config.json
  • v2/extension-brave-config.json
  • v2/extension-bravemv3-config.json
  • v2/extension-chrome-config.json
  • v2/extension-chromemv3-config.json
  • v2/extension-config.json
  • v2/extension-edg-config.json
  • v2/extension-edge-config.json
  • v2/extension-edgmv3-config.json
  • v2/extension-firefox-config.json
  • v2/extension-safarimv3-config.json
  • v2/ios-config.json
  • v2/macos-config.json
--- v2/macos-config.json
+++ v2/macos-config.json
@@ -6341,12 +6341,8 @@
             },
             "exceptions": [],
             "state": "disabled"
         },
-        "voiceSearch": {
-            "exceptions": [],
-            "state": "disabled"
-        },
         "webCompat": {
             "exceptions": [
                 {
                     "domain": "lastpass.com",
v1/windows-config.json
--- v1/windows-config.json
+++ v1/windows-config.json
@@ -243,9 +243,10 @@
             ],
             "settings": {
                 "disabledCMPs": [
                     "generic-cosmetic"
-                ]
+                ],
+                "enableIfMainWorldIsSupported": "enabled"
             },
             "state": "disabled"
         },
         "autofill": {
@@ -6338,12 +6339,8 @@
             },
             "exceptions": [],
             "state": "disabled"
         },
-        "voiceSearch": {
-            "exceptions": [],
-            "state": "disabled"
-        },
         "webCompat": {
             "exceptions": [],
             "state": "disabled"
         },
v2/windows-config.json
--- v2/windows-config.json
+++ v2/windows-config.json
@@ -243,11 +243,13 @@
             ],
             "settings": {
                 "disabledCMPs": [
                     "generic-cosmetic"
-                ]
+                ],
+                "enableIfMainWorldIsSupported": "enabled"
             },
-            "state": "disabled"
+            "state": "disabled",
+            "minSupportedVersion": "0.50.0"
         },
         "autofill": {
             "exceptions": [
                 {
@@ -6331,12 +6333,8 @@
             },
             "exceptions": [],
             "state": "disabled"
         },
-        "voiceSearch": {
-            "exceptions": [],
-            "state": "disabled"
-        },
         "webCompat": {
             "exceptions": [],
             "state": "disabled"
         },
latest
v3/android-config.json
--- v3/android-config.json
+++ v3/android-config.json
@@ -7076,16 +7076,8 @@
             },
             "exceptions": [],
             "state": "disabled"
         },
-        "voiceSearch": {
-            "exceptions": [],
-            "state": "enabled",
-            "settings": {
-                "minVersion": 33,
-                "excludedManufacturers": []
-            }
-        },
         "webCompat": {
             "exceptions": [],
             "state": "enabled",
             "settings": {
v3/extension-brave-config.json (11 more)
  • v3/extension-brave-config.json
  • v3/extension-bravemv3-config.json
  • v3/extension-chrome-config.json
  • v3/extension-chromemv3-config.json
  • v3/extension-config.json
  • v3/extension-edg-config.json
  • v3/extension-edge-config.json
  • v3/extension-edgmv3-config.json
  • v3/extension-firefox-config.json
  • v3/extension-safarimv3-config.json
  • v3/ios-config.json
  • v3/macos-config.json
--- v3/macos-config.json
+++ v3/macos-config.json
@@ -6348,12 +6348,8 @@
             },
             "exceptions": [],
             "state": "disabled"
         },
-        "voiceSearch": {
-            "exceptions": [],
-            "state": "disabled"
-        },
         "webCompat": {
             "exceptions": [
                 {
                     "domain": "lastpass.com",
v3/windows-config.json
--- v3/windows-config.json
+++ v3/windows-config.json
@@ -243,11 +243,13 @@
             ],
             "settings": {
                 "disabledCMPs": [
                     "generic-cosmetic"
-                ]
+                ],
+                "enableIfMainWorldIsSupported": "enabled"
             },
-            "state": "disabled"
+            "state": "disabled",
+            "minSupportedVersion": "0.50.0"
         },
         "autofill": {
             "exceptions": [
                 {
@@ -6338,12 +6340,8 @@
             },
             "exceptions": [],
             "state": "disabled"
         },
-        "voiceSearch": {
-            "exceptions": [],
-            "state": "disabled"
-        },
         "webCompat": {
             "exceptions": [],
             "state": "disabled"
         },

@github-actions
Copy link

Generated file outputs:

legacy
fingerprinting.json (29 more)
  • fingerprinting.json
  • protections.json
  • trackers-unprotected-temporary.txt
  • trackers-whitelist-temporary.txt
  • v1/android-config.json
  • v1/extension-brave-config.json
  • v1/extension-bravemv3-config.json
  • v1/extension-chrome-config.json
  • v1/extension-chromemv3-config.json
  • v1/extension-config.json
  • v1/extension-edg-config.json
  • v1/extension-edge-config.json
  • v1/extension-edgmv3-config.json
  • v1/extension-firefox-config.json
  • v1/extension-safarimv3-config.json
  • v1/ios-config.json
  • v1/macos-config.json
  • v2/android-config.json
  • v2/extension-brave-config.json
  • v2/extension-bravemv3-config.json
  • v2/extension-chrome-config.json
  • v2/extension-chromemv3-config.json
  • v2/extension-config.json
  • v2/extension-edg-config.json
  • v2/extension-edge-config.json
  • v2/extension-edgmv3-config.json
  • v2/extension-firefox-config.json
  • v2/extension-safarimv3-config.json
  • v2/ios-config.json
  • v2/macos-config.json

⚠️ File is identical

v1/windows-config.json
--- v1/windows-config.json
+++ v1/windows-config.json
@@ -243,9 +243,13 @@
             ],
             "settings": {
                 "disabledCMPs": [
                     "generic-cosmetic"
-                ]
+                ],
+                "enableIfMainWorldIsSupported": {
+                    "state": "enabled",
+                    "minSupportedVersion": "0.50.0"
+                }
             },
             "state": "disabled"
         },
         "autofill": {
v2/windows-config.json
--- v2/windows-config.json
+++ v2/windows-config.json
@@ -243,11 +243,16 @@
             ],
             "settings": {
                 "disabledCMPs": [
                     "generic-cosmetic"
-                ]
+                ],
+                "enableIfMainWorldIsSupported": {
+                    "state": "enabled",
+                    "minSupportedVersion": "0.50.0"
+                }
             },
-            "state": "disabled"
+            "state": "disabled",
+            "minSupportedVersion": "0.50.0"
         },
         "autofill": {
             "exceptions": [
                 {
latest
v3/android-config.json (12 more)
  • v3/android-config.json
  • v3/extension-brave-config.json
  • v3/extension-bravemv3-config.json
  • v3/extension-chrome-config.json
  • v3/extension-chromemv3-config.json
  • v3/extension-config.json
  • v3/extension-edg-config.json
  • v3/extension-edge-config.json
  • v3/extension-edgmv3-config.json
  • v3/extension-firefox-config.json
  • v3/extension-safarimv3-config.json
  • v3/ios-config.json
  • v3/macos-config.json

⚠️ File is identical

v3/windows-config.json
--- v3/windows-config.json
+++ v3/windows-config.json
@@ -243,11 +243,16 @@
             ],
             "settings": {
                 "disabledCMPs": [
                     "generic-cosmetic"
-                ]
+                ],
+                "enableIfMainWorldIsSupported": {
+                    "state": "enabled",
+                    "minSupportedVersion": "0.50.0"
+                }
             },
-            "state": "disabled"
+            "state": "disabled",
+            "minSupportedVersion": "0.50.0"
         },
         "autofill": {
             "exceptions": [
                 {

Copy link
Contributor

@SlayterDev SlayterDev left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the detailed description 👍

@SlayterDev SlayterDev merged commit 10d72c6 into main Aug 22, 2023
@SlayterDev SlayterDev deleted the boli/re-enable-autoconsent-windows branch August 22, 2023 21:05
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.

2 participants