jeedom-zigate / jeedom-plugin-zigate

Plugin Jeedom for ZiGate key.
MIT License
29 stars 12 forks source link

Problème syncronisation Matériel sunricher #122

Closed nono2032 closed 5 years ago

nono2032 commented 5 years ago

Le sunricher est compatible ZiGate mais il n'y a pas de retour automatique. il faut faire la demander manuellement. Du coup, si vous souhaitez de la réactivité, il faut faire un polling serré sur le plugin. ZG9101SAC-HP. est-ce possible de l'ajouter ? Merci et salutations

guardrails[bot] commented 5 years ago

:warning: We detected security issues in this pull request:

Hard-Coded Secrets (1) - [/resources/zigated/zigated.py:195](https://github.com/Jeedom-Zigate/jeedom-plugin-zigate/blob/9ac4e75bdeb5cb934ff6530c7d20f29c14c18a83/resources/zigated/zigated.py#L195) More info on how to fix Hard-Coded Secrets in [General](http://docs.guardrails.io/vulnerabilities/general/hard-coded_secrets.html?utm_source=ghpr). ---
Insecure Network Communication (1) - [/resources/zigated/zigated.py:64](https://github.com/Jeedom-Zigate/jeedom-plugin-zigate/blob/9ac4e75bdeb5cb934ff6530c7d20f29c14c18a83/resources/zigated/zigated.py#L64) More info on how to fix Insecure Network Communication in [Python](http://docs.guardrails.io/vulnerabilities/py/insecure_network_communication.html?utm_source=ghpr). ---
Insecure Use of Dangerous Function (1) - [/resources/zigated/zigated.py:232](https://github.com/Jeedom-Zigate/jeedom-plugin-zigate/blob/9ac4e75bdeb5cb934ff6530c7d20f29c14c18a83/resources/zigated/zigated.py#L232) More info on how to fix Insecure Use of Dangerous Function in [Python](http://docs.guardrails.io/vulnerabilities/py/insecure_use_of_dangerous_function.html?utm_source=ghpr).

Happy with the results? Give your feedback.

doudz commented 5 years ago

woo, the PR est beaucoup trop gros et casse beaucoup de chose, merci de nettoyer tout ça avant et refaire un PR sur la branche develop

llaumgui commented 5 years ago

D'une manière générale faut pas faire de PR sur master, le travail est sur develop.