Cron job fails to execute correctly - getStorageUsed.sh about borgwarehouse HOT 7 CLOSED

Its one of the commands that will be gone with the 2.0 release. (from the sudoers list)

For now, it should be fine. It doesn't really change anything from before.

It could be made more secure by changing it to the following, if you really want to:

/usr/bin/du -s /var/borgwarehouse/* | sed 's/\/var\/borgwarehouse\///' | jc --du

borgwarehouse ALL=(ALL) NOPASSWD: /usr/bin/du -s /var/borgwarehouse/[[\:xdigit\:]]*

That way you can only query the subfolders under the /var/borgwarehouse path.

from borgwarehouse.

Thanks for your feedback. I will test on Ubuntu 22.04.2 to try to reproduce your bug.

from borgwarehouse.

@Aterfax Can you tell me if it's Ubuntu with desktop env or if it's ubuntu server image without gui ? Thank you

from borgwarehouse.

Fairly sure it is Ubuntu server with XFCE on it, but is used 99% of the time headlessly.

from borgwarehouse.

Thank you, it is to allow me to reproduce the error as well as possible. I'm doing some tests today and I'll let you know.

from borgwarehouse.

I was able to reproduce the bug. It's still strange... especially since the script runs normally when I launch it manually, it causes an error only when executed by node.

I think this is a version specific problem with jc, I don't see any other explanation:

• Debian 11 - jc 1.14.3 ✅
• Fedora 37 - jc 1.22.5 ✅
• Ubuntu Server 22.04 - jc 1.17.3 ❌

Thank you @Aterfax for looking for the solution and sharing it ! I'm studying it in a branch if the solution has no impact on Debian and Fedora. If it's ok, I'll modify the code to improve the compatibility with Ubuntu Server.

@p-rintz Can you validate the change in sudo permissions please ? 🙏

from borgwarehouse.

Thank you @p-rintz I'll test it. And if it is problematic it will be removed in v2 anyway so it is not very important.

from borgwarehouse.