search

mendhak / waveshare-epaper-display

At-a-glance dashboard for Raspberry Pi with a Waveshare ePaper 7.5 Inch HAT. Date/Time, Weather, Alerts, Google/Outlook Calendar
https://code.mendhak.com/raspberrypi-epaper-dashboard/
438 stars 65 forks source link

Security: Do not require sudoers rights in order to run the script #15

Closed lifeofbrian closed 3 years ago

lifeofbrian commented 3 years ago

I notice that the script has separate scripts that require sudoers rights, and as a result the environment variables are not passed along to those scripts, so for example the display.py logic to handle the v2 of the ePaper display doesn't work

lifeofbrian commented 3 years ago

I will submit a pull request on this one

mendhak commented 3 years ago

Thanks! Just testing it now

mendhak commented 3 years ago

Looks good, no sudo required. I've merged it.
In case it doesn't work after merging, it's probably because some files are owned by root.
Do an ls -lah then make sure to sudo chown pi:pi xxxx any files that are owned by root.

mendhak commented 3 years ago

Thanks, closing issues now.

lifeofbrian commented 3 years ago

Great!