-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Brave pkg installer creates BraveSoftware folder as root #30546
Comments
@spylogsster will this fix anyone currently experiencing this issue or will this only prevent the problem going forward? Marking as |
Per discussion in above thread, this prevents the problem going forward, it will not fix anyone currently experiencing this issue. From @spylogsster via https://bravesoftware.slack.com/archives/C01826CM3J9/p1690469928520929?thread_ts=1690214773.687379&cid=C01826CM3J9:
Note - removing the BraveSoftware folder will remove your profile. Proceed with caution on either option if you choose to use one. |
Verified with
Reproduced the issue using STR from description.
Case 1 - Install 1.57.x (Beta) using refcode tagged .pkg and install 1.58.x (Nightly) using .dmg - PASSED Covers:
Case 2 - Install version on 1.57.x without this fix and then update to the latest - PASSED Covers:
Case 3 - Re-run Case 1 but as non-Admin user - PASSED Covers:
pre-req: be logged in to machine as non-admin user
Case 4 - Spot check on-demand updates - PASSED Covers:
Admin user:
Non-Admin user:
|
The above requires |
Verified with
Reproduced the issue using STR from description.
Existing User Case - attempt to manually fix install currently experiencing this issue - PASSED Note, per internal discussions, simply upgrading to latest version will not fix the issue it just prevents it from occurring in the future. However, following the below steps (installing latest from brave.com over your existing install) does fix the permissions issue.
Case 1 - Install 1.56.x (RC) using refcode tagged .pkg and install 1.58.x (Nightly), 1.57.x (Beta) using .dmg - Partially PASSED Note, since we only have one RC which contains this fix, unable to check updating on Release from one version with the fix to another version with the fix. This was checked prior to uplift on both Nightly and Beta though. Covers:
Case 2 - Install version on 1.56.x (Release) without this fix and then update to the latest (using background updates) - PASSED Covers:
Case 3 - Re-run Case 1 but as non-Admin user - Partially PASSED Note, since we only have one RC which contains this fix, unable to check updating on Release from one version with the fix to another version with the fix. This was checked prior to uplift on both Nightly and Beta though. Covers:
pre-req: be logged in to machine as non-admin user
Case 4 - Spot check on-demand updates - Partially PASSED Note, as mentioned above, since there's only one RC with this fix, unable to verify on-demand updates going from a version with the fix to a version with the fix. Covers:
Admin user:
Non-Admin user:
Case 1 - Install 1.56.x (RC with fix) using refcode tagged .pkg and update to another version with the fix using background updates - Remainder PASSED Covers:
Case 3 - Re-run Case 1 but as non-Admin user - Remainder PASSED Covers:
pre-req: be logged in to machine as non-admin user
Case 4 - Spot check on-demand updates - Remainder PASSED Covers:
Admin user:
Non-Admin user:
|
Verified
|
Example | Example |
---|---|
Non-Admin user:
- Removed previous installs and profiles as follows:
- Deleted all Brave instances from
Applications
folder - Deleted
BraveSoftware
folder from~/Library/Application Support
- Deleted
BraveSoftware
folder from~/Library/Caches
- Don't need to worry about other items this time or restarting the machine
- Deleted all Brave instances from
- Downloaded Release (1.56.14) universal .pkg file (used GH to get this, could also use brave.com) and changed
-universal
to-BRV002
to simulate it being tagged for ref code - Installed Release from .pkg
- Launch Release with logs:
/Applications/Brave\ Browser.app/Contents/MacOS/Brave\ Browser --args --enable-logging=stderr --update-feed-url=https://updates.bravesoftware.com/sparkle/Brave-Browser/test-arm64/appcast.xml
- Confirm Release launches fine, no issues
- Quickly close Release before background update is found
- Relaunch Release and navigate to brave://settings/help
- Confirm update is found
- Relaunch and confirm Release is updated to RC version (follow any prompts you receive)
Example | Example | Example | Example |
---|---|---|---|
Verified with
Reproduced the issue using STR from description.
Existing User Case - attempt to manually fix install currently experiencing this issue - PASSED Note, per internal discussions, simply upgrading to latest version will not fix the issue it just prevents it from occurring in the future. However, following the below steps (installing latest from brave.com over your existing install) does fix the permissions issue.
Case 1 - Install 1.56.x (RC) using refcode tagged .pkg and install 1.58.x (Nightly), 1.57.x (Beta) using .dmg - Partially PASSED Note, since we only have one RC which contains this fix, unable to check updating on Release from one version with the fix to another version with the fix. This was checked prior to uplift on both Nightly and Beta though. Covers:
Case 2 - Install version on 1.56.x (Release) without this fix and then update to the latest (using background updates) - PASSED Covers:
Case 3 - Re-run Case 1 but as non-Admin user - Partially PASSED Note, since we only have one RC which contains this fix, unable to check updating on Release from one version with the fix to another version with the fix. This was checked prior to uplift on both Nightly and Beta though. Covers:
pre-req: be logged in to machine as non-admin user
Case 4 - Spot check on-demand updates - Partially PASSED Note, as mentioned above, since there's only one RC with this fix, unable to verify on-demand updates going from a version with the fix to a version with the fix. Covers:
Admin user:
Non-Admin user:
Case 1 - Install 1.56.x (RC with fix) using refcode tagged .pkg and update to another version with the fix using background updates - Remainder PASSED Covers:
Case 3 - Re-run Case 1 but as non-Admin user - Remainder PASSED Covers:
pre-req: be logged in to machine as non-admin user
Case 4 - Spot check on-demand updates - Remainder PASSED Covers:
Admin user:
Non-Admin user:
|
Verification
|
Example | Example |
---|---|
Non-Admin user:
Removed previous installs and profiles as follows:
- Deleted all Brave instances from
Applications
folder - Deleted
BraveSoftware
folder from~/Library/Application Support
- Deleted
BraveSoftware
folder from~/Library/Caches
- Don't need to worry about other items this time or restarting the machine
- Downloaded a 1.53.99 Beta (this is 1.56.x code base) universal .pkg file (used GH to get this) and added
-BRV002
to simulate it being tagged for ref code - Installed Beta from .pkg
- Launch Beta with logs:
/Applications/Brave\ Browser\ Beta.app/Contents/MacOS/Brave\ Browser\ Beta --args --enable-logging=stderr
- Confirm Beta launches fine, no issues
- Quickly close Beta before background update is found
- Relaunch Beta and navigate to brave://settings/help
- Confirm update is found
- Relaunch and confirm Release is updated to latest
Example | Example | Example |
---|---|---|
Beta channel
Admin user:
- Removed previous installs and profiles as follows:
- Deleted all Brave instances from
Applications
folder - Deleted
BraveSoftware
folder from~/Library/Application Support
- Deleted
BraveSoftware
folder from~/Library/Caches
- Deleted
com.brave.*
folders from~/Library/Caches
- Deleted
com.brave.*
files from~/Library/Preferences
- Restarted my machine
- Deleted all Brave instances from
- Downloaded Beta (1.57.17) universal .pkg file (used GH to get this) and added
-BRV002
to simulate it being tagged for ref code - Installed Beta from .pkg
- Launch Beta with logs:
/Applications/Brave\ Browser\ Beta.app/Contents/MacOS/Brave\ Browser\ Beta --args --enable-logging=stderr
- Confirm Beta launches fine, no issues
- Quickly close Beta before background update is found
- Relaunch Beta and navigate to brave://settings/help
- Confirm update is found
- Relaunch and confirm Beta is updated to latest
Example | Example |
---|---|
Non-Admin user:
Removed previous installs and profiles as follows:
- Deleted all Brave instances from
Applications
folder - Deleted
BraveSoftware
folder from~/Library/Application Support
- Deleted
BraveSoftware
folder from~/Library/Caches
- Don't need to worry about other items this time or restarting the machine
- Downloaded a 1.53.99 Beta (this is 1.56.x code base) universal .pkg file (used GH to get this) and added
-BRV002
to simulate it being tagged for ref code - Installed Beta from .pkg
- Launch Beta with logs:
/Applications/Brave\ Browser\ Beta.app/Contents/MacOS/Brave\ Browser\ Beta --args --enable-logging=stderr
- Confirm Beta launches fine, no issues
- Quickly close Beta before background update is found
- Relaunch Beta and navigate to brave://settings/help
- Confirm update is found
- Relaunch and confirm Beta is updated to latest
Example | Example | Example |
---|---|---|
Nightly channel
Admin user:
- Removed previous installs and profiles as follows:
- Deleted all Brave instances from
Applications
folder - Deleted
BraveSoftware
folder from~/Library/Application Support
- Deleted
BraveSoftware
folder from~/Library/Caches
- Deleted
com.brave.*
folders from~/Library/Caches
- Deleted
com.brave.*
files from~/Library/Preferences
- Restarted my machine
- Deleted all Brave instances from
- Downloaded Beta (1.57.17) universal .pkg file (used GH to get this) and added
-BRV002
to simulate it being tagged for ref code - Installed Beta from .pkg
- Launch Beta with logs:
/Applications/Brave\ Browser\ Beta.app/Contents/MacOS/Brave\ Browser\ Beta --args --enable-logging=stderr
- Confirm Beta launches fine, no issues
- Quickly close Beta before background update is found
- Relaunch Beta and navigate to brave://settings/help
- Confirm update is found
- Relaunch and confirm Nightly is updated to latest
Example | Example |
---|---|
Non-Admin user:
Removed previous installs and profiles as follows:
- Deleted all Brave instances from
Applications
folder - Deleted
BraveSoftware
folder from~/Library/Application Support
- Deleted
BraveSoftware
folder from~/Library/Caches
- Don't need to worry about other items this time or restarting the machine
- Downloaded a 1.53.99 Beta (this is 1.56.x code base) universal .pkg file (used GH to get this) and added
-BRV002
to simulate it being tagged for ref code - Installed Beta from .pkg
- Launch Beta with logs:
/Applications/Brave\ Browser\ Beta.app/Contents/MacOS/Brave\ Browser\ Beta --args --enable-logging=stderr
- Confirm Beta launches fine, no issues
- Quickly close Beta before background update is found
- Relaunch Beta and navigate to brave://settings/help
- Confirm update is found
- Relaunch and confirm Nightly is updated to latest
Example | Example | Example |
---|---|---|
Description
Pkg installer sets root permission to
~/Library/Application Support/BraveSoftware
folder.drwxr-xr-x 3 root staff 96 May 24 13:10 BraveSoftware
.It doesnt allow browsers from other channels to create their profiles.
Steps to Reproduce
Actual result:
Browser crashed on start
Expected result:
Browser launched successfully
Reproduces how often:
100%
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
The text was updated successfully, but these errors were encountered: