search

salbahra / Sprinklers-PHP

A mobile interface for the OpenSprinkler irrigation device written in PHP. This program is no longer developed and is superseded by the Javascript app: https://github.com/salbahra/Sprinklers.
http://albahra.com/
30 stars 13 forks source link

ERR - Error communicating with OpenSprinkler. Please check your pasword is correct. #30

Closed brianmccarty closed 10 years ago

brianmccarty commented 10 years ago

After I login, if I attempt to toggle the manual mode I am getting the following error flashing on my window. I see the manual mode toggling on and off in a loop.

Error communicating with OpenSprinkler. Please check your pasword is correct ![opensprinkler__manual_control]

(https://f.cloud.github.com/assets/5996164/2291557/4e444246-a041-11e3-87c7-677d66439314.jpg)

salbahra commented 10 years ago

Absolutely right. I found the bug and should have it fixed soon. Thanks!

On Friday, February 28, 2014, brianmccarty notifications@github.com wrote:

After I login, if I attempt to toggle the manual mode I am getting the following error flashing on my window. I see the manual mode toggling on and off in a loop.

Reply to this email directly or view it on GitHubhttps://github.com/salbahra/OpenSprinkler-Controller/issues/30 .

salbahra commented 10 years ago

This should be resolved in commit https://github.com/salbahra/OpenSprinkler-Controller/commit/8bcb7f697bc65f348ef433e236fa868df6287fd6

Please let me know if this addresses it.

Thanks!

brianmccarty commented 10 years ago

This is still an issue. The loop does not occur in the updated version, but it still refuses to enter manual mode, with the same error popup.

salbahra commented 10 years ago

The issue is because your password is incorrect. The device default password is "opendoor" (without the quotes). Please check the password is correct. I will try and update the error message to reflect this.

brianmccarty commented 10 years ago

When I first setup the system months ago, I changed the password with the interval app. All had been functioning until the other day.
I went back and changed the password in the interval app back to opendoor and you are correct that it functions now. Where in the Sprinklers app, or config can I update the passwords to match what is in the interval app

salbahra commented 10 years ago

The app doesn't require the password for anything except executing changes on the OpenSprinkler and thus why the password issue was missed for so long.

Currently, you have to modify the config.php file and change the password there. Maybe in the future I will expose this as an option within the settings. 

#Set OpenSprinkler password
$os_pw = "opendoor";
brianmccarty commented 10 years ago

Excellent, thanks for the quick response. and a great app.

natosys commented 10 years ago

Looks like there may be an issue in the system (possibly not the web app) where it cannot handle long passwords. At the moment I cannot get it to work with a complex password of around 25 characters.

salbahra commented 10 years ago

Interesting, I will take a look at this and see if I can replicate it. I doubt it's a limitation of the storage engine (localStorage).