hotzenklotz / picobrew-server

A reverse engineered server for the proprietary PicoBrew Zymatic protocol
MIT License
61 stars 16 forks source link

Loading recipe to Zymatic fails. #6

Closed danyou closed 4 years ago

danyou commented 4 years ago

The Zymatic screen goes blank after attempting to load any recipe. Either cleaning or brewing recipe.

pico server.txt

/API/SyncUSer?user=00000000000000000000000000000000&machine=xxxxxxxx0000

danyou commented 4 years ago

I think the error is the Zymatic is now sending /API/SyncUSer?user= instead of /API/SyncUser?user. The uppercase S now in User appears to be the issue. The following request generates the error 404

/API/SyncUSer?user=00000000000000000000000000000000&machine=xxxxxxxx0000

danyou commented 4 years ago

picobrew_recipe.py", line 32, in serialize AttributeError: 'PicoBrewRecipe' object has no attribute 'id' no attribute id.txt

hotzenklotz commented 4 years ago

Thanks for reporting these issues. I have not tested the server with the machine since a few brews. Do you mind opening a pull request with your suggested changes. Thanks

@danyou Do you mind sharing the failing recipe as well? Thanks.

danyou commented 4 years ago

Just opened the pull request. Somehow the filename hash doesn't get created and there is no id. The below are copied from recipes folder after a successful import via web interface. (added .txt for github)

Not_Yet-ie_recipe_xml.xml.txt

Party_Porter_recipe_xml.xml.txt

picobrew_recipe line 20 generates id from file name. self.id = get_hash(filename) picobrew_recipe line 33 self.id errors with object has no attribute 'id'

fotoapparat commented 4 years ago

@danyou, I have the server up and running on my computer and can't for the life of me get my computer to reroute PicoBrew.com to the server. Did you alter your hosts file or use another method? If so, would you mind sharing?

danyou commented 4 years ago

I had the Zymatic connected to my laptop. I used ICS just to have a DHCP server for the Zymatic. I have the following in my host file: 192.168.137.1 picobrew.com

Also used app.run(host="0.0.0.0", port=80)

dns
danyou commented 4 years ago

Just as a note of caution: If the Zymatic faults and the screen goes blank, DON'T leave it powered on. I did and the circulating pump shut off and the heater stayed on. A tube in the glycol loop ruptured.

Blackjackbuyer commented 4 years ago

This is the file I was forwarded. Please double check it for anything nasty before opening it I haven’t looked at it yet just received it tonight.

On Mon, 4 May 2020 at 22:07, danyou notifications@github.com wrote:

Just as a note of caution: If the Zymatic faults and the screen goes blank, DON'T leave it powered on. I did and the circulating pump shut off and the heater stayed on. A tube in the glycol loop ruptured.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/hotzenklotz/picobrew-server/issues/6#issuecomment-623706403, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALOAZ2LXJS56TTBSZE4EA6LRP4VCDANCNFSM4MX2YDQQ .

--

  • Colm McGoldrick
Blackjackbuyer commented 4 years ago

Also.

http://picobrewcontent.blob.core.windows.net/firmware/zseries/zseries_0_0_116.bin

On Mon, 4 May 2020 at 22:28, Colm McGoldrick colm.mcgoldrick@gmail.com wrote:

This is the file I was forwarded. Please double check it for anything nasty before opening it I haven’t looked at it yet just received it tonight.

On Mon, 4 May 2020 at 22:07, danyou notifications@github.com wrote:

Just as a note of caution: If the Zymatic faults and the screen goes blank, DON'T leave it powered on. I did and the circulating pump shut off and the heater stayed on. A tube in the glycol loop ruptured.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/hotzenklotz/picobrew-server/issues/6#issuecomment-623706403, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALOAZ2LXJS56TTBSZE4EA6LRP4VCDANCNFSM4MX2YDQQ .

--

  • Colm McGoldrick

--

  • Colm McGoldrick
hotzenklotz commented 4 years ago

@danyou I pushed a fix for the id issue (to master) that should take care of the bug. Please try again and let me know if the fix is working.

danyou commented 4 years ago

I can confirm the recipe is now sent to the Zymatic (with id).

hotzenklotz commented 4 years ago

Great!