Brew has not been added to sudoers.d

[MattRogowski]

I keep getting this, despite having run sudo valet trust in the past, and running it again now. I have also run sudo chmod +r /private/etc/sudoers.d/brew to no avail. Is there anything else to try?

MacOS 12.4, M1 Max, Valet 3.1.9

I've found #23 from a year and a half ago, the issue still remains.

[nicoverbruggen]

Hi @MattRogowski!

What does your /private/etc/sudoers.d/brew file look like? On M1, it should reference /opt/homebrew/bin/brew, like this:

Cmnd_Alias BREW = /opt/homebrew/bin/brew *
%admin ALL=(root) NOPASSWD:SETENV: BREW

This can come up if you're running PHP Monitor natively for M1 while you have Homebrew running with Rosetta. In that case, you may want to try running PHP Monitor using Rosetta (please consult the FAQ for more info about that, under the section "I've got two Homebrew installations on my Apple Silicon Mac, can I choose which installation to use with PHP Monitor?").

[jamesh-purr]

@nicoverbruggen this is still an issue. I have tried numerous times to get this to work. I tried with/without rosetta ticked. I have tried installing under arm64/x86.

I either get brew has not been added to sudoers file or that homebrew binary can't be found

[nicoverbruggen]

Perhaps if you turn on verbose logging I might be able to help you identify this issue!

Verbose logging is disabled by default. You can start extra verbose logging by running: touch ~/.config/phpmon/verbose and restarting PHP Monitor, until it informs you about the issue. Then, you can find the latest log in: ~/.config/phpmon/last_session.log. (You can remove the ~/.config/phpmon/verbose file when done, all it does is tell PHP Monitor to turn on verbose logging for this very purpose.)

Please attach the last_session.log file here and I'll take a look at the output. If some unexpected terminal output is included in some of the commands I expect to run, then that could cause this issue.

(As an aside, did you see my video from the FAQ about setting up your system for PHP Monitor?)

[jamesh-purr]

Hey @nicoverbruggen thanks for getting back to me, its much appreciated it. I have attached it. I have checked whats inside and its basically saying brew don't exist in the location it is looking for but it does. I have attached a screenshot.

last_session.log

[nicoverbruggen]

@jamesh-purr Ah, I see what the issue is. PHP Monitor expects to find the path /usr/local/bin/brew in the sudoers.d/brew file.

The expected brew and valet paths are listed in the log:

[ARCH] The user is running PHP Monitor with the architecture: x86_64
[OK] `/usr/local/bin/brew` exists
[OK] `/usr/local/bin/php` exists

If you can adjust the sudoers.d/brew file to look like this:

Cmnd_Alias BREW = /usr/local/bin/brew *
%admin ALL=(root) NOPASSWD:SETENV: BREW

... then PHP Monitor should start up correctly. Let me know if that's actually the case!

(Technically speaking I think your setup is OK as it is, so I've also opened #251 to ensure that the check is a little bit more lenient. For PHP Monitor to work correctly, you will need to edit the file as I noted above. Assuming you can confirm this fixes the problem, hopefully I can also resolve this with a fix, so manually editing this file won't be needed in the future!)

[jamesh-purr]

Hey @nicoverbruggen thanks for your time. Unfortunately that still don't seem to be working. I did the following,

sudo nano /private/etc/sudoers.d/brew
Cmnd_Alias BREW = /usr/local/bin/brew *
%admin ALL=(root) NOPASSWD:SETENV: BREW

saved the file.

I have also changed /private/etc/sudoers.d/valet with the above.

The issue persists :( . I greatly appreciate your time on this. I find php monitor so useful which is why i want to get this resolved haha.

Do I need to restart anything?
last_session.log

[nicoverbruggen]

@jamesh-purr

Well yes, the /private/etc/sudoers.d/valet file needs to be altered too, you were a bit too eager to edit that one with the same contents. It should look like this on your system:

Cmnd_Alias VALET = /usr/local/bin/valet *
%admin ALL=(root) NOPASSWD:SETENV: VALET

The logs says we did get a bit further than last time, so this should do the trick!

[jamesh-purr]

Hey @nicoverbruggen yup your right i did jump gun. We got a bit further

<~~~~~~~~~~~~~~~~~~~~~~~
$ /bin/sh --noprofile -norc --login -c "export PATH=/usr/local/bin:$PATH && which node"

[OUT]:

[OK] `env: node` issue does not apply

<~~~~~~~~~~~~~~~~~~~~~~~
$ /bin/sh --noprofile -norc --login -c "export PATH=/usr/local/bin:$PATH && valet --version"

[OUT]:
[ERR]:
/bin/sh: valet: command not found
~~~~~~~~~~~~~~~~~~~~~~~~>

[OK] `no global composer issues

<~~~~~~~~~~~~~~~~~~~~~~~
$ /bin/sh --noprofile -norc --login -c "export PATH=/usr/local/bin:$PATH && valet --version"

[OUT]:
[ERR]:
/bin/sh: valet: command not found
~~~~~~~~~~~~~~~~~~~~~~~~>

[FAIL] `valet --version` was loaded
[P] deinit: <PHP_Monitor.BetterAlertVC: 0x600000344690>.deinit

If i use valet --version

jameshayes@Jamess-MacBook-Pro-2 local % valet --version
Laravel Valet 4.1.1

[nicoverbruggen]

@jamesh-purr Hmm... Valet (valet) needs to be in /usr/local/bin as well.

You can symlink it there and that should fix the issue:

sudo ln -s /usr/local/homebrew/bin/valet /usr/local/bin

Did you recently (re)install Homebrew, and did you do so in a different location from the default? There's a chance that the default path for binaries has been changed with a recent release of Homebrew and that's causing this issue.

(If Homebrew's default installation directory on Intel has been modified, I'll need to make some adjustments in #251 to make sure that /usr/local/homebrew/bin is also checked. Thanks for bringing this to my attention!)

[nicoverbruggen]

I've actually made a quick build that should be able to handle this altered location. If you're feeling adventurous, you can try it out: phpmon-dev.zip

NOTE: with this new DEV build, you shouldn't need the symlinks and you should be able to restore the sudoers files to what they were before, which means:

Cmnd_Alias BREW = /usr/local/homebrew/bin/brew *
%admin ALL=(root) NOPASSWD:SETENV: BREW

Cmnd_Alias VALET = /usr/local/homebrew/bin/valet *
%admin ALL=(root) NOPASSWD:SETENV: VALET

[jamesh-purr]

Hey @nicoverbruggen

Great thank you for such a quick response/fix! It seems it is all working now as expected! Again thank you for your time and effort.