search

SignalK / set-system-time

Signal K Node server plugin to set system time on time data from GPS
Apache License 2.0
3 stars 6 forks source link

Check passwordless capacity for date strictly aganst the date command. #5

Closed FredericGuilbault closed 5 years ago

FredericGuilbault commented 5 years ago

This fix https://github.com/SignalK/set-system-time/issues/4

Previously, the ability of using the date command without password was checked aganst the echo command. Witch would fail in restrictive context where only sudo date command is allowed password less and not sudo echo

Sudoers entry for reference : 

signalk ALL=(ALL) NOPASSWD: /bin/date

Note: The command have been tested the a terminal only.

tkurki commented 5 years ago

Please add the sudoers entry to README.

FredericGuilbault commented 5 years ago

2nd commit added

tkurki commented 5 years ago

Thanks! Published as @signalk/set-system-time@1.3.0.