-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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 reading /etc/chromium/policies #19052
Comments
brave/brave-core#7277 was supposed to fix this, but it seems the override isn't working. My system doesn't have an |
Further testing:
Previously I thought it was reading from |
I think the problem is that the override fails if the directory doesn't exist. We call base::PathService::Override(
chrome::DIR_POLICY_FILES,
base::FilePath(FILE_PATH_LITERAL("/etc/brave/policies"))); Which calls if (!PathExists(file_path) && !CreateDirectory(file_path))
return false; and the override is never inserted into the map, so subsequent calls use the original path. Since an ordinary user can't create a new |
PathService::Override() requires that the new target path exists, and tries to create it if it doesn't. Since the user doesn't have permission to create `/etc/brave` the override fails if an administrator hadn't already installed something there, and the original `/etc/chromium` path for policy files remains active. Current uses of this path are few and seem to be robust against the directory not existing, so this seems a safer fix than patching the upstream default. Closes brave/brave-browser#19052
Verified
on Fedora 35 Workstation
Confirmed there's no longer the Reproduced the issue using
|
Description
On Fedora linux systems, brave loads policy settings from
/etc/chromium/policies/managed
which is populated by thefedora-chromium-config
package, installed by default on Fedora Workstation targets to support the project's gssapi sign-on system.This results in a 'Your browser is managed by your organization' warning at the top of the settings page, which may be alarming for private users.
Steps to Reproduce
sudo dnf install brave-browser-beta
Actual result:
"Your browser is managed by your organization" appears at the top of
brave://settings
.Expected result:
Since this makes tracking easier, I would expect brave to not load this chrome/chromium-specific configuration data.
Reproduces how often:
Only tried on my current laptop, which is a pretty recent install. Removing the
fedora-chromium-config
package also removes the warning from within brave.Brave version (brave://version info)
Brave 1.32.76 Chromium: 95.0.4638.54 (Official Build) beta (64-bit)
Revision d31a821ec901f68d0d34ccdbaea45b4c86ce543e-refs/branch-heads/4638@{#871}
OS Linux
Version/Channel Information:
The text was updated successfully, but these errors were encountered: