tagyoureit / nodejs-Pentair

An application to read/write to Pentair pool controllers.
14 stars 6 forks source link

Thank you. I have change the adapter I was using and now I have success. However, my pool appear off when is actually on. Chemistry is also unavailable. The pool pump, however, is showing activity correctly. Any ideas? #65

Closed mzuniga51 closed 2 years ago

mzuniga51 commented 2 years ago

Thank you. I have change the adapter I was using and now I have success. However, my pool appear off when is actually on. Chemistry is also unavailable. The pool pump, however, is showing activity correctly. Any ideas?

Screen Shot 2021-10-07 at 21 34 05 Screen Shot 2021-10-07 at 21 37 44

s?

Originally posted by @mzuniga51 in https://github.com/tagyoureit/nodejs-Pentair/issues/64#issuecomment-938313015

mzuniga51 commented 2 years ago

Also, I noticed that nothing appears under controller model, neither the expansion card is detected (should it?

Screen Shot 2021-10-07 at 21 58 56

)

tagyoureit commented 2 years ago

Can you open a packet capture (see wiki)? We should be detecting both of these.

mzuniga51 commented 2 years ago

Thanks for the help. I'm getting this when trying to generate the packet capture

Screen Shot 2021-10-08 at 13 41 31

:

tagyoureit commented 2 years ago

Delete your ./data*.json files and follow the command line instructions.

mzuniga51 commented 2 years ago

Hello. I reinstall the to programs to send you a log where I start on clean slate and I found the following:

  1. Installation of the controller presented some "vulnerabilities". Here attached is the audit report. I then proceed to do npm audit fix, which corrected all vulnerabilities except five.
  2. Neither the run of the controller nor that of the panel were really completed. I've included both logs showing, I imagine, what went wrong. Still the web interface seem up, with the errors I explained (pool is shown as off when is really off and a few other inconsistencies with the Intellicenter).
  3. I'm attaching the zip file you requested with the packet capture.

I appreciate your help and the work you have done for our community very much. Thank you in advance

Manuel

On Fri, Oct 8, 2021 at 4:54 PM tagyoureit @.***> wrote:

Delete your ./data*.json files and follow the command line https://github.com/tagyoureit/nodejs-poolcontroller/wiki/How-to-capture-all-packets-for-issue-resolution#command-line instructions.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/tagyoureit/nodejs-Pentair/issues/65#issuecomment-939156143, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV2HSS4ZEWAB6LIJXBB4P53UF5ZB7ANCNFSM5FSV5XLQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

    === npm audit security report ===    

Run npm update ansi-regex --depth 5 to resolve 3 vulnerabilities

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint > strip-ansi > ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint > table > string-width > strip-ansi > ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint > table > strip-ansi > ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘

Run npm update path-parse --depth 7 to resolve 9 vulnerabilities

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint-plugin-import > │ │ │ eslint-import-resolver-node > resolve > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-plugin-import │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-plugin-import > eslint-import-resolver-node > resolve │ │ │ > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint-plugin-import > read-pkg-up > │ │ │ read-pkg > normalize-package-data > resolve > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-plugin-import │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-plugin-import > read-pkg-up > read-pkg > │ │ │ normalize-package-data > resolve > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-plugin-node [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-plugin-node > resolve > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ grunt [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ grunt > grunt-cli > liftup > rechoir > resolve > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint-plugin-import > resolve > │ │ │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-plugin-import │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-plugin-import > resolve > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service in path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ grunt [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ grunt > grunt-cli > liftup > resolve > path-parse │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-hj48-42vr-x3v9 │ └───────────────┴──────────────────────────────────────────────────────────────┘

Run npm update color-string --depth 5 to resolve 1 vulnerability

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Regular Expression Denial of Service (ReDOS) │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ color-string │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ winston │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ winston > @dabh/diagnostics > colorspace > color > │ │ │ color-string │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-257v-vj4p-3w2h │ └───────────────┴──────────────────────────────────────────────────────────────┘

Run npm update lodash --depth 4 to resolve 8 vulnerabilities

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Command Injection in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > inquirer > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-35jh-r3h4-6jhm │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Command Injection in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-35jh-r3h4-6jhm │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Command Injection in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > table > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-35jh-r3h4-6jhm │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Command Injection in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ node-ssdp │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ node-ssdp > async > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-35jh-r3h4-6jhm │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Prototype Pollution in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > inquirer > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-p6mc-m468-83gw │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Prototype Pollution in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-p6mc-m468-83gw │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Prototype Pollution in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > table > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-p6mc-m468-83gw │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Prototype Pollution in lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ node-ssdp │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ node-ssdp > async > lodash │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-p6mc-m468-83gw │ └───────────────┴──────────────────────────────────────────────────────────────┘

┌──────────────────────────────────────────────────────────────────────────────┐ │ Manual Review │ │ Some vulnerabilities require your attention to resolve │ │ │ │ Visit https://go.npm.me/audit-guide for additional guidance │ └──────────────────────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=5.0.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > inquirer > string-width > │ │ │ strip-ansi > ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=5.0.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > inquirer > strip-ansi > │ │ │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=5.0.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > strip-ansi > ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=5.0.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ eslint-config-promise │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ eslint-config-promise > eslint > table > string-width > │ │ │ strip-ansi > ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Moderate │ Inefficient Regular Expression Complexity in │ │ │ chalk/ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=5.0.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ serialport │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ serialport > @serialport/bindings > prebuild-install > │ │ │ npmlog > gauge > wide-align > string-width > strip-ansi > │ │ │ ansi-regex │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://github.com/advisories/GHSA-93q8-gq69-wqmw │ └───────────────┴──────────────────────────────────────────────────────────────┘ found 26 vulnerabilities (18 moderate, 8 high) in 651 scanned packages run npm audit fix to fix 21 of them. 5 vulnerabilities require manual review. See the full report for details.

tagyoureit commented 2 years ago

First, do NOT run the npm audit fix commands. If something gets updated outside of what we specify I cannot guarantee that it can work.

Second, Github doesn't work for attaching files in email. Please log into the github.com browser and upload the logs/files manually. They were never attached.

Third, how did you end up opening a case in this repository? It was defunct 5+ years ago. You should be using http://github.com/tagyoureit/nodejs-poolController.

mzuniga51 commented 2 years ago

I'm a complete noob and a couple weeks ago I wasn't even aware that GitHub was a thing. I'm moving the thread to the new repository. Thanks for bearing with me.