chestm007 / linux_thermaltake_riing

Python driver and daemon to control thermaltake Riing fans and pumps
GNU General Public License v2.0
59 stars 25 forks source link

Use safe_load() instead of load() #56

Closed mlgomez closed 3 months ago

mlgomez commented 1 year ago

I'd like to start off by saying I'm a little bit new to pull requests so I apologize in advance if any etiquette is being breached.

safe_load() would add the benefit of loading the YAML configuration without allowing for any circumstance where python objects could arbitrarily be loaded into it. It's a small change that does not, from what I can see, impact the function of this TTRGB daemon.

Alternatively, _yaml.fullload(cfg) could be used to reduce the number of arguments being passed here to just one.