Forum Replies Created
-
Posts
-
> “anything I don’t specify reverts to defaults” — I am surprised by this
OK I didn’t check everything – perhaps an assumption. I didn’t set watering level ‘wl’ (or anything after it), and it get set to 0 if I didn’t include it in the request. I wasn’t quite sure what the rest of the parameters were.
I filed the support ticket with the configuration attached.
Using the app does resolve the issue, but the app is missing options like the Cloud settings.
@Ray Thanks will do. It’s pretty much vanilla OS3 install (reset, set password, set location, then set zimmerman).
I did a bit of network tracing and the submit button is causing a “413 The request was too large”
Here’s the request that fails on submit:
http://192.168.1.21/co?pw=xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx&tyy=2022&tmm=11&tdd=24&thh=5&tmi=58&ttt=1671861536&tz=16&loc=34.04053%2C-117.18228&lg=1&mas=0&mton=0&mtof=0&mas2=0&mton2=0&mtof2=0&ext=0&sdt=0&uwt=1&wto=%22h%22%3A100%2C%22t%22%3A100%2C%22r%22%3A100%2C%22bh%22%3A30%2C%22bt%22%3A70%2C%22br%22%3A0%2C%22pws%22%3A%22%22%2C%22key%22%3A%22%22&wl=100&sn1o=1&fpr0=100&fpr1=0&sn1on=0&sn1of=0&sn2o=1&sn2on=0&sn2of=0&ifkey=&ife=0&dname=My+OpenSprinkler&dim=30&hp0=80&hp1=0&devid=0&ntp1=0&ntp2=0&ntp3=0&ntp4=0&ip1=192&ip2=168&ip3=1&ip4=21&gw1=192&gw2=168&gw3=1&gw4=1&subn1=255&subn2=255&subn3=255&subn4=0&dns1=192&dns2=168&dns3=1&dns4=31&dhcp=1&ntp=1&ipas=0&sar=0&sn1t=0&sn2t=0&_=1671861532470It also happens with the “Monthly” setting. Manual, ETo and Auto Rain Delay all work fine.
I tried removing a lot some of the remaining parameters after the
wtoparameter and that worked – well sort of – anything I don’t specify reverts to defaults. I found the same issue when trying to set the cloud key up as well.I’m unable to update the Weather Adjustment method to Zimmerman. Every time I pick that option and hit Submit, the saving spinner will show up and just stay there, never completing. If I refresh the page the changes were not committed.
At the very least, do POST instead of GET, so the password doesn’t end up in the url. It’s a little too obvious that way. I assume Arduino can handle a POST.
In any case the ‘correct’ way to do http is that GET only allows getting data, but POST/PUT/DELETE etc performs operations (like turning a valve on).
Overall the protocol OpenSprinkler is using is a little “weird” and not really following the typical REST patterns. It’s definitely the first time I’ve seen the need for bit-wise operators to parse input/output. I assume this is to save the amount of data being sent, but I doubt that extreme compression is worth it over just doing some normal obvious spelled-out set of JSON operators.
Btw… I get you can fix the password issue in the apps. It still needs fixing on the device itself, since it’s built-in webpages doesn’t properly encode parameters either.
Aaaah! I can’t believe I didn’t think of this (considering the password is in the URL in clear text and wasn’t encoded) 😯 😮 😕
Thank you! So this is a problem not only in the app but also in the built-in website. Weird that it works for some of it though
Awesome work. I can’t even begin to imagine all the horrors of getting one app to run on all devices.
You’re welcome. I get the point of building one app for them all, but unfortunately it also means a really poor user experience and an app that doesn’t feel like it’s made for the device.
I had started making my own home automation dashboard app. I could probably take on making an entire app for instead that will use the platform a little more. I put it on GitHub so far: https://github.com/dotMorten/OpenSprinklerNet Don’t mind the app piece yet… it’s ugly and just to quickly test it for now.
@semag wrote:
Kris-
In Southern California (I know both San Diego and Santa Barbara do this) they release a “percentage” value weekly that is based on ET data.You can get the SoCal water percentage values here:
http://www.mwdh2o.com/watering_index/watering_index_test.txtFirst line is daily, then weekly, then monthly adjustment. Would be nice to be able to pull the automatically and adjust based on the daily value.
