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

Removing all folders/files associated with ads/rewards from "session_profiles/Tor Profile" #11645

Closed
kjozwiak opened this issue Sep 10, 2020 · 3 comments
Labels
bug closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. feature/ads feature/rewards OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include security

Comments

@kjozwiak
Copy link
Member

Description

If the user has been using rewards and has opened a Tor Window, they'll have several files/folders/logs associated with ads/rewards within session_profiles/Tor Profile. We've stopped this from happening via brave/brave-core#6418 but haven't covered the cases of removing files that might have already existed. Example of files that might already be in session_profiles/Tor Profile but are not being removed after updating to a version with brave/brave-core#6418 merged.

* ~/Library/Application Support/BraveSoftware/Brave-Browser-Nightly/Default/session_profiles/Tor Profile/ads_service
* ~/Library/Application Support/BraveSoftware/Brave-Browser-Nightly/Default/session_profiles/Tor Profile/rewards_service
* ~/Library/Application Support/BraveSoftware/Brave-Browser-Nightly/Default/session_profiles/Tor Profile/publisher_info_db

Similar to the following issue:

Steps to Reproduce

  1. launch a version of brave before Fixes Brave Rewards service is associated to Tor Profiles brave-core#6418 was merged
  2. enable rewards
  3. go into /BraveSoftware/Brave-Browser-Nightly/Default/session_profiles/Tor Profile/ and you should see several folders/files associated with ads/rewards as mentioned above.
  4. update to a new version and you'll notice the folders/files associated with ads/rewards haven't been removed

Actual result:

brave/brave-core#6418 doesn't remove the folders/files that already exists within /BraveSoftware/Brave-Browser-Nightly/Default/session_profiles/Tor Profile/ that might be associated with rewards/ads.

Expected result:

Should remove all the folders/files associated with rewards/ads from /BraveSoftware/Brave-Browser-Nightly/Default/session_profiles/Tor Profile/

Reproduces how often:

100% reproducible using the above STR.

Brave version (brave://version info)

Brave | 1.16.4 Chromium: 85.0.4183.102 (Official Build) nightly (64-bit)
-- | --
Revision | ffe848af6a5df4fa127e2929331116b7f9f1cb30-refs/branch-heads/4183@{#1770}
OS | macOS Version 10.15.6 (Build 19G73)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

CCing @diracdeltas @rebron @NejcZdovc @jsecretan @tmancey for prioritization.

@GeetaSarvadnya
Copy link

Reproduced in windows 10 x64 - 1.13.86

@btlechowski
Copy link

Reproduced on

Brave 1.13.86 Chromium: 85.0.4183.102 (Official Build) (64-bit)
Revision ffe848af6a5df4fa127e2929331116b7f9f1cb30-refs/branch-heads/4183@{#1770}
OS Ubuntu 18.04 LTS

@diracdeltas diracdeltas added the priority/P3 The next thing for us to work on. It'll ride the trains. label Sep 22, 2020
@NejcZdovc NejcZdovc removed the priority/P2 A bad problem. We might uplift this to the next planned release. label Sep 24, 2020
@NejcZdovc NejcZdovc removed their assignment Sep 24, 2020
@tmancey tmancey changed the title [Desktop] removing all folders/files associated with ads/rewards from "session_profiles/Tor Profile" Removing all folders/files associated with ads/rewards from "session_profiles/Tor Profile" Dec 5, 2020
@tmancey tmancey added the closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. label Aug 24, 2021
@tmancey
Copy link
Contributor

tmancey commented Aug 24, 2021

Closing issue as stale. @kjozwiak can you please confirm you are happy for me to close this issue

@tmancey tmancey closed this as completed Aug 24, 2021
@tmancey tmancey added this to Ads Jun 10, 2024
@tmancey tmancey moved this to Done in Ads Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. feature/ads feature/rewards OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include security
Projects
Archived in project
Development

No branches or pull requests

7 participants