vzlogger icon indicating copy to clipboard operation
vzlogger copied to clipboard

Support for OpenSSL 3.0+

Open maxberger opened this issue 2 years ago • 8 comments

This patch adds support for openssl 3.0 and later.

It also moves the sha1 calculation into its own file to enable testing.

Code is heavily inspired by https://gitlab.isc.org/isc-projects/kea/-/merge_requests/1582/diffs#0592d076cdc8b76110bb63b48b37ca2416ad31c4

maxberger avatar Jun 29 '22 20:06 maxberger

For the mocketermap error see pr #528

maxberger avatar Jun 29 '22 21:06 maxberger

This should also fix #535 then.

BoxedBrain avatar Aug 21 '22 11:08 BoxedBrain

Leute.

Es ist absolut unklar, wie ihr das hinbekommt. Könnte bitte $irgendwer einfach mal das master Repo lieb machen, dass das mit einem frischen Ubuntu funktioniert oder erklären, wie man den Patch einspielt. Nicht jeder versteht ein RTFM auf anhieb. Bitte einfach mal eine Zeile mehr --verbose kann doch nicht schaden.

Das https://github.com/volkszaehler/vzlogger ist sauber gecloned und bekomme auch die o.a. Fehler auf nem frischen jammy.

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.1 LTS
Release:        22.04
Codename:       jammy

$ cd ~/vzlogger
$ git checkout -p 530
No changes.

Es tut sich nichts. Bitte. Vielleicht könnte sich einer von euch erbarmen und sagen, wie dieser Patch eingespielt wird. Danke.

DonKingMat avatar Sep 25 '22 16:09 DonKingMat

Updated, please review again. Tests fail currently due to unrelated issues, I will create another pull request for those.

maxberger avatar Sep 25 '22 19:09 maxberger

Leute.

Es ist absolut unklar, wie ihr das hinbekommt. Könnte bitte $irgendwer einfach mal das master Repo lieb machen, dass das mit einem frischen Ubuntu funktioniert oder erklären, wie man den Patch einspielt. Nicht jeder versteht ein RTFM auf anhieb. Bitte einfach mal eine Zeile mehr --verbose kann doch nicht schaden.

Das https://github.com/volkszaehler/vzlogger ist sauber gecloned und bekomme auch die o.a. Fehler auf nem frischen jammy.

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.1 LTS
Release:        22.04
Codename:       jammy

$ cd ~/vzlogger
$ git checkout -p 530
No changes.

Es tut sich nichts. Bitte. Vielleicht könnte sich einer von euch erbarmen und sagen, wie dieser Patch eingespielt wird. Danke.

Hi @DonKingMat,

leider ist es nicht ganz so einfach - Aktuell braucht man zum übersetzen diesen Patch, dann noch #528, und leider nochmal einen den ich gerade erstelle (da gtest gerade kaputt ist).

Da es sich bei vzlogger um freiwillige Arbeit handelt, dauert es immer etwas mit den reviews. Da auch ich nur in meiner Freizeit mithelfe, dauert es auch immer, bis ich die gewuenschten Aenderungen machen kann, damit der patch dann auch eingepflegt wird. Daher kann ich nur abwarten empfehlen, falls du nicht mithelfen kannst.

Max

maxberger avatar Sep 25 '22 19:09 maxberger

Ich Danke Dir Max!

Absolut großartiger Support! Das ist ja das schöne am #Neuland ... Jeder kann was und einige bringen sich mit ihrem Skill ein, so wie Du! Ja leider bleibt es auch nach Deinen Updates bei einem

/home/pi/vzlogger/src/api/MySmartGrid.cpp:559:45: error: ‘HMAC_CTX* HMAC_CTX_new()’ is deprecated: Since OpenSSL 3.0 [-Werror=deprecated-declarations]

Aber sehr sehr großes Danke schon mal für Deine super schnelle Reaktion.

Also abwarten und der alte Pi mit openssl 1.1.1n-0+deb10u3 muss dann eben weiter laufen und vzlogger wird eben noch nicht migriert auf den CM4.

Issue #530 ist on watchlist ... Bin ja nicht der einzige mit dem Problem ... Irgendwer wird es sicher bald richten und in Ermangelung an C Kenntnissen, muss ich leider leider passen ....

DonKingMat avatar Sep 25 '22 19:09 DonKingMat

This needs #541 and #528 to pass the tests.

maxberger avatar Sep 25 '22 19:09 maxberger

Ich Danke Dir Max!

Absolut großartiger Support! Das ist ja das schöne am #Neuland ... Jeder kann was und einige bringen sich mit ihrem Skill ein, so wie Du! Ja leider bleibt es auch nach Deinen Updates bei einem

/home/pi/vzlogger/src/api/MySmartGrid.cpp:559:45: error: ‘HMAC_CTX* HMAC_CTX_new()’ is deprecated: Since OpenSSL 3.0 [-Werror=deprecated-declarations]

Aber sehr sehr großes Danke schon mal für Deine super schnelle Reaktion.

Also abwarten und der alte Pi mit openssl 1.1.1n-0+deb10u3 muss dann eben weiter laufen und vzlogger wird eben noch nicht migriert auf den CM4.

Issue #530 ist on watchlist ... Bin ja nicht der einzige mit dem Problem ... Irgendwer wird es sicher bald richten und in Ermangelung an C Kenntnissen, muss ich leider leider passen ....

Ich habe einen branch erzeugt, der alle meine patches beinhaltet: https://github.com/maxberger/vzlogger/tree/all

Damit sollte es funktionieren, bis alles gemerged ist.

maxberger avatar Sep 25 '22 19:09 maxberger