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

Remove refcode from P3A reports #21460

Closed
rillian opened this issue Mar 3, 2022 · 4 comments · Fixed by brave/brave-core#12478
Closed

Remove refcode from P3A reports #21460

rillian opened this issue Mar 3, 2022 · 4 comments · Fixed by brave/brave-core#12478

Comments

@rillian
Copy link

rillian commented Mar 3, 2022

Description

Per discussion today, we no longer use the refcode field of the P3A measurement associated metadata, and so we should stop reporting it.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 1, 2022

Verification PASSED on


Brave | 1.38.75 Chromium: 100.0.4896.60 (Official Build) beta (64-bit)
-- | --
Revision | 6a5d10861ce8de5fce22564658033b43cb7de047-refs/branch-heads/4896@{#875}
OS | Windows 10 Version 21H2 (Build 19044.1586)

Clean profile

  • Confirmed messages to the p3a.brave.com endpoint list the none in the refcode field (Note: the cmd echo CesyWJC+2AQCEkADLElOLHdpbng2NC1iYywxLjM4Ljc1LGJldGEsMjIxNCwyMjE0LG5vbmUsMQAAAAAAAAAAAAAAAAAAAAAAAAAA | base64 -d needs to be executed and get the platform specific base64-coded measurement string from Contents p3a.brave.com endpoint tab in Charles)
1.37.x 1.38.x
image (10) image
  • Confirmed message to the p3a-json.brave.com endpoint doesn't contain a refcode field in 1.38.x
1.37.x 1.38.x
image image (5)

Upgrade profile

Case 1: Install 1.37.x and then upgrade the profile to 1.38.x and close the browser and then set the system date to tomorrow and relaunch the browser, after relaunching the browser for ~10 mins or so, the refcode strings are removed from the http://p3a-json.brave.com/ endpoints as the refcode removal code is merged in 1.38.x

Case 2: Install 1.37.x and collect metrics with refcode and move system date to mid of next Sunday evening (i.e 10th April 12:01:00 AM which falls on Monday, on which weekly reports are collected) and then upgrade the profile to 1.38.x and collect the metrics without refcode. In Monday's report, we should be able to see both with and without refcode metrics

Case 3: Install 1.37.x and upgrade the profile to 1.38.x and close the browser, If user has not used or opened the profile for a week and if we set the clock ahead by a week and relaunch brave, the report is picking up the metrics without refcode.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 1, 2022

@rillian Refcode field is removed from p3a.brave.com and p3a-json.brave.com endpoints only? Browser usage pings are still showing up refcode in the response. Is that ok?

image

@rillian
Copy link
Author

rillian commented Apr 1, 2022

Yes, that's ok. The change only affects P3A.

@Uni-verse
Copy link
Contributor

Verification completed on Samsung GS 21, Samsung Tab S7 using

Brave	1.38.107 Chromium: 101.0.4951.41 (Official Build) (64-bit) 
Revision	93c720db8323b3ec10d056025ab95c23a31997c9-refs/branch-heads/4951@{#904}
OS	Android 12;

Fresh Profile:

  • Confirmed requests made to the p3a-json.brave.com endpoint should not contain a refcode field at all.

Upgrade Profile:

  • Setting up a profile with refcode and then upgrading to 1.38 will send metrics without the refcode .
  • Collect metrics with refcode, upgrade the profile and then move the clock one week ahead will send new reports without refcode.
1.37 1.38
Screen Shot 2022-04-26 at 9 33 02 PM Screen Shot 2022-04-26 at 9 27 56 PM

avinassh pushed a commit to avinassh/brave-browser-hardening that referenced this issue May 29, 2022
 - Added the ability to redirect an AMP page to its canonical non-AMP version. ([#20458](brave/brave-browser#20458))
 - Added "Select network" field in the Brave Wallet "Add custom asset" modal. ([#21656](brave/brave-browser#21656))
 - Added isBraveWallet property to window.ethereum. ([#21949](brave/brave-browser#21949))
 - Added the ability to copy Brave Wallet transaction hash from "Transactions" on the "Portfolio" page. ([#19886](brave/brave-browser#19886))
 - Added confirmation screen after transaction is approved for Brave Wallet. ([#21733](brave/brave-browser#21733))
 - Added bottom padding to the Brave Wallet swap container. ([#21340](brave/brave-browser#21340))
 - Added loading skeletons for various areas within Brave Wallet. ([#21339](brave/brave-browser#21339))
 - Implemented balance updater interval for Brave Wallet. ([#21304](brave/brave-browser#21304))
 - Updated design for Brave Shields. ([#18630](brave/brave-browser#18630))
 - Updated IPFS node to go-ipfs v0.12.0. ([#21219](brave/brave-browser#21219))
 - Updated adblocking to execute all cosmetic scriptlets even if one fails. ([#21932](brave/brave-browser#21932))
 - Updated wording on Brave Wallet backup reminder banner. ([#21312](brave/brave-browser#21312))
 - Updated Brave Wallet swap capability to prevent unnecessarily encountering "Too many decimal places" error. ([#21527](brave/brave-browser#21527))
 - Updated URL for Brave Wallet Ropsten Test Faucet network. ([#21240](brave/brave-browser#21240))
 - Updated Brave Wallet password field to be cleared when an incorrect password is entered. ([#21010](brave/brave-browser#21010))
 - Updated "Portfolio" page of Brave Wallet to return all EVM network tokens. ([#20854](brave/brave-browser#20854))
 - Updated UI to increase spacing between "Assets" and "Transactions" on "Accounts" subview page of Brave Wallet. ([#21532](brave/brave-browser#21532))
 - Updated link text color to blue for Brave Wallet onboarding screen. ([#20302](brave/brave-browser#20302))
 - Updated alignment of options available from the Brave Wallet "More" menu. ([#19680](brave/brave-browser#19680))
 - Renamed "Monthly Contributions" to "Monthly Tips" for Brave Rewards. ([#20564](brave/brave-browser#20564))
 - Removed redirect after adding an account to Brave Wallet from the portfolio asset details screen. ([#19138](brave/brave-browser#19138))
 - Removed referral code from P3A reports. ([#21460](brave/brave-browser#21460))
 - Fixed unexpected storage cleanup on site close when "Clear cookies and site data when you close all windows" is enabled under brave://settings/cookies. ([#22493](brave/brave-browser#22493))
 - Fixed unblocked cookie consent dialogs on some websites when the brave://flags/#brave-adblock-cookie-list-default is enabled. ([#22032](brave/brave-browser#22032))
 - Fixed inability to solve Brave Rewards CAPTCHA in certain cases. ([#22180](brave/brave-browser#22180))
 - Fixed inability to import MetaMask wallet to Brave Wallet. ([#22492](brave/brave-browser#22492))
 - Fixed failed second sign request (eth_signTypedData_v4) for Loopring.io L2 activation when using Brave Wallet. ([#22070](brave/brave-browser#22070))
 - Fixed data on "Portfolio" page of Brave Wallet not being properly displayed due to parsing error. ([#21941](brave/brave-browser#21941))
 - Fixed broken placeholder icons for ERC721 tokens on Brave Wallet "Account" page. ([#21549](brave/brave-browser#21549))
 - Fixed "JSON data is not expected" when trying to import a crypto wallet into Brave Wallet in certain cases. ([#19526](brave/brave-browser#19526))
 - Fixed Brave Wallet network selector list being partially blurred on page load. ([#21423](brave/brave-browser#21423))
 - Fixed Brave Wallet "Add Custom Asset" modal incorrectly being populated with previously added custom token details. ([#21337](brave/brave-browser#21337))
 - Fixed EIP-1559 gas oracle incorrectly firing on non-EIP-1559 networks for Brave Wallet. ([#21107](brave/brave-browser#21107))
 - Fixed Brave Wallet percentage selector remaining selected when input value was updated. ([#20315](brave/brave-browser#20315))
 - Fixed Speedreader icon incorrectly disappearing in certain cases. ([#18243](brave/brave-browser#18243))
 - Fixed inability to exit reader mode when Speedreader is disabled. ([#18242](brave/brave-browser#18242))
 - Upgraded Chromium to 101.0.4951.41. ([#22431](brave/brave-browser#22431)) ([Changelog for 101.0.4951.41](https://chromium.googlesource.com/chromium/src/+log/100.0.4896.127..101.0.4951.41?pretty=fuller&n=1000))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants