Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pin PyCryptodomex to version 3.9.4 #575

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tiefpunkt
Copy link

The latest release of PyCryptodomex seems to be missing the Cryptodome.Util submodule, making the SmartConfig process fail. It works with the 3.9.4 version though.

The latest release of PyCryptodomex seems to be missing the Cryptodome.Util submodule. It works with the 3.9.4 version though.
@kueblc
Copy link
Collaborator

kueblc commented Mar 27, 2020

Thanks for your PR!

I don't see anything about this change in the PyCryptodome repo, and the Util module does seem to still be present in the source.

I will take a closer look when a get a chance, but feel free to add more info when available.

@tiefpunkt
Copy link
Author

I did the normal installation procedures, and when starting the update, I got an error message along the lines of "no module named Cryptodome.Util". I then also did the PyCryptodome selftest ("python -m Cryptodome.SelfTest"), which failed with a similar message. Had to do it in a linux live installation, so I don't have the exact error messages anymore, sorry.

So it's definitly a PyCryptodome issue, but going back to an older version of PyCryptodomex at least let me flash my wifi plugs, so I thought I'd share that ;)

@bakman2
Copy link

bakman2 commented Apr 8, 2020

I needed to use this to make it work.

Actually the install_req script does not seem to work properly at all for me, all required modules were not installed, although it stated it did. manually installed and succeeded.

@kueblc
Copy link
Collaborator

kueblc commented Apr 8, 2020

Interesting @bakman2, could you please share more details? What OS? Were any errors reported?

@bakman2
Copy link

bakman2 commented Apr 8, 2020

Interesting @bakman2, could you please share more details? What OS? Were any errors reported?

This was on a pi running octopi (Linux octopi 4.19.75-v7+ #1270 not sure what was done to the os specifically).

Installing, no errors reported. (sorry, i removed all directories)

Running start_flash revealed "no module Cryptodome", manually installed, restart flash, missing tornado, repeat, missing sslpkg.
Then got an error about sslpsk.git, could not write in /tmp: error 1 None. sudo, no success.

Ending up switching to root and installed all manually (forgot paho) but could flash.

@alonevino
Copy link

alonevino commented Aug 4, 2020

I landed here because I had the same issue mentioned in #583
I can also confirm what @bakman2 is saying. I can add more details, hopefully useful:

  • in smarthack-psk is mentioned that the module sslpsk is missing
  • in smarthack-udp is mentioned that the module Cryptodome is missing
  • in smarthack-web is mentioned that the module tornado is missing
    Further more running install_prereq.sh there was error "no matching distribution found for praho-mqtt" due to issues with certificates.
    I'm running Ubuntu 20.04

@bakman2 how did you sorted your mentioned issues out ?

@kxn25
Copy link

kxn25 commented Dec 6, 2020

And what can i do if i hav sonoffs already running?
the Flashing seems to get interuptet with my sonoff devices?
can i change the mqtt port?

I used my Pi3 and my tablet for the confim network.
I think the problem was that my Android has trustet Wifi thing...
With my old Tablet there where no problem and after few attemps its uploaded the firmware.

@kueblc
Copy link
Collaborator

kueblc commented Dec 6, 2020

@kxn25 this project only supports Tuya devices, not Sonoff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants