-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Confirm first party domain blocking (#14134) off in Release Channel build #15149
Comments
@brave/legacy_qa we will need a 1.23.x RC to check this issue. |
Basically what @LaurenWags mentioned above but here's more detailed STR/Cases. I picked pub.chez.com but any of the ones that @LaurenWags mentioned above can be used as well.
|
Removing |
Waiting on @iefremov to push these changes into Griffen. |
Worked with @iefremov and went through the following on
|
Unblocking as brave/brave-variations#14 has been merged into production. There's still one issue re: new installs needing to restart the browser. Still TBD if this will get addressed by |
Verification passed on OnePlus 6T with Android 10 running 1.23.69 x64 build
},
"variations_compressed_seed": "H4sIAAAAAAAAAL3OPQ/BQACA4TskOEtzSUUsGtPFcOmXatemFiN/wF3vSqMfUiWxEv9bG4MUs3d+hwdp3DTn3DVtx+BV0nEcL1qEOg+51EPLZvgGkeoX7CKDPGVx5id5eNiUZ3F14WqCusuM8UQKBXCMlM9vpaFeEJ9eh+AYfx8D1A1kxM5JqYC1Ou57OjWoadGZ1iaAQNIinS3E9wqxPO5lKguWbMq8YDv5AyFqxOfXQIAa8XU0EEOMXK9GGHRhNRSPPypGDQXQWm/HFBpPuM9RyLgBAAA=",
"variations_country": "IN",
"variations_crash_streak": 0,
"variations_failed_to_fetch_seed_streak": 0,
"variations_last_fetch_time": "13262696710793465",
"variations_permanent_consistency_country": [ "90.1.23.69", "IN" ],
"variations_safe_compressed_seed": "H4sIAAAAAAAAAL3OPQuCQACA4TsLrGuRgyJakiZpEO/KrzV0caw/0OmdFVmGadBa9L9TGkJtbni3d3iQG5scOMTklBluavMyKljy2KDVsS4QRdfEDouEqYzfhpSd2OK+SNDpu8oLfHRhMkeyfWZgIroAQI6X5BSrqeYfr5+Ahxu1jgGRPxKxIcgWsh5O+a+hEpwt9rnY0oEFN0rpbiJ8lwr/sxUlkLNnkacZ24geCV4jmV0OACtE6aogRRo5bIYhuL2qK1x8V45oCqNLXMYPkDcntuxi4AQAA",
"variations_safe_seed_date": "13262694873000000",
"variations_safe_seed_fetch_time": "13262694876299507",
"variations_safe_seed_locale": "en-US",
"variations_safe_seed_permanent_consistency_country": "IN",
"variations_safe_seed_session_consistency_country": "IN",
"variations_safe_seed_signature": "MEUCIQD0oqynrJRGgJNLQUhz/SzgdVJNRw8WU7YmNgaS5x6BOAIgHetkiltEjq17sF/0+aTx2wnCssKhnuCCImSSWrs5nTU=",
"variations_seed_date": "13262696711000000",
"variations_seed_signature": "MEQCIAOkK3lKIetQ3piryhqQox29I1N501knG8pcpXbic695AiAK7lwK1IP07f/mrLZXdo8o3wjblJyZ7pWeMNnpUsy5gQ=="
}
Verification passed on Samsung Tab A with Android 10 running 1.23.69 x64 build
},
"variations_compressed_seed": "H4sIAAAAAAAAAL3OPQ/BQACA4TskOEtzSUUsGtPFcOmXatemFiN/wF3vSqMfUiWxEv9bG4MUs3d+hwdp3DTn3DVtx+BV0nEcL1qEOg+51EPLZvgGkeoX7CKDPGVx5id5eNiUZ3F14WqCusuM8UQKBXCMlM9vpaFeEJ9eh+AYfx8D1A1kxM5JqYC1Ou57OjWoadGZ1iaAQNIinS3E9wqxPO5lKguWbMq8YDv5AyFqxOfXQIAa8XU0EEOMXK9GGHRhNRSPPypGDQXQWm/HFBpPuM9RyLgBAAA=",
"variations_country": "IN",
"variations_crash_streak": 0,
"variations_failed_to_fetch_seed_streak": 0,
"variations_last_fetch_time": "13262694348918483",
"variations_permanent_consistency_country": [ "90.1.23.69", "IN" ],
"variations_safe_compressed_seed": "H4sIAAAAAAAAAL3OPQuCQACA4TsLrGuRgyJakiZpEO/KrzV0caw/0OmdFVmGadBa9L9TGkJtbni3d3iQGscOMTklBluavMyKljy2KDVsS4QRdfEDouEqYzfhpSd2OK+SNDpu8oLfHRhMkeyfWZgIroAQI6X5BSrqeYfr5+Ahxu1jgGRPxKxIcgWsh5O+a+hEpwt9rnY0oEFN0rpbiJ8lwr/sxUlkLNnkacZ24geCV4jmV0OACtE6aogRRo5bIYhuL2qK1x8V45oCqNLXMYPkDcntuxi4AQAA",
"variations_safe_seed_date": "13262688324000000",
"variations_safe_seed_fetch_time": "13262688352684664",
"variations_safe_seed_locale": "en-US",
"variations_safe_seed_permanent_consistency_country": "IN",
"variations_safe_seed_session_consistency_country": "IN",
"variations_safe_seed_signature": "MEUCIQD0oqynrJRGgJNLQUhz/SzgdVJNRw8WU7YmNgaS5x6BOAIgHetkiltEjq17sF/0+aTx2wnCssKhnuCCImSSWrs5nTU=",
"variations_seed_date": "13262694323000000",
"variations_seed_signature": "MEQCIAOkK3lKIetQ3piryhqQox29I1N501knG8pcpXbic695AiAK7lwK1IP07f/mrLZXdo8o3wjblJyZ7pWeMNnpUsy5gQ=="
} Note: I did have to restart the browser multiple times and visit the test sites before the entire variation logs showed up. This is a known issue for now |
Verification passed on
Verified test plan from #15149 (comment) on Brave release and variation seed production server Verified that the interstitials page wasn't being displayed
Local state:
Verification passed on
Verified test plan from #15149 (comment) on Brave release and variation seed production server Verified that the interstitials page wasn't being displayed
Local state:
|
Closing as @pilgrim-brave fixed/merged brave/brave-core#8507 into |
Adding |
Removing Verified
Tested and ensured that 1st-party domain blocking is OFF by successfully loading:
Verification PASSED on
Verification passed on
|
Placeholder to confirm #14134 is off by default on Release Channel builds.
Example:
Visit one of the sites listed in #14134 (comment) such as:
1-1ads.com
actionsplash.com
pub.chez.com
Confirm that the site loads as expected, this means that first-party domain blocking is off by default. This is the expected outcome at this time.
However, if the site is blocked then first-party domain blocking is on by default. This should not be the case for Release Channel builds at this time.
The text was updated successfully, but these errors were encountered: