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

AVM: Bugfix #791

Merged
merged 1 commit into from
Aug 10, 2023
Merged

AVM: Bugfix #791

merged 1 commit into from
Aug 10, 2023

Conversation

sisamiwe
Copy link
Contributor

@sisamiwe sisamiwe commented Aug 7, 2023

  • FritzHome: Bugfix renewing SID in case SID has expired
  • CallMonitor: bugfix join listing thread
  • CallMonitor: reconnect if connection is lost
  • bump to 2.0.9

- FritzHome: Bugfix renewing SID in case SID has expired
- CallMonitor: bugfix join listing thread
- CallMonitor: reconnect if connection is lost
- bump to 2.0.9
@sisamiwe sisamiwe requested review from Morg42, aschwith and psilo909 and removed request for Morg42 August 7, 2023 18:45
@psilo909
Copy link
Contributor

psilo909 commented Aug 8, 2023

hmm not sure why the tests are failing here. ERROR says something with logging. I will test the changes later today!

@msinn
Copy link
Member

msinn commented Aug 8, 2023

Ich schau da später mal rein. Ich habe gestern am Test Environment geschraubt und hatte einige Themen, bis der Core wieder lief.

@msinn
Copy link
Member

msinn commented Aug 8, 2023

Der PR wurde genau in dem Zeitraum gestellt, als die Tests im Core mit dem selben Problem fehlschlugen. Der Fix der Tests im Core wurde erst ca. eine Stunde später committed. Deshalb vermute ich, dass bei weiteren Pull Requests für Plugins die Tests wieder normal funktionieren.

@aschwith
Copy link
Contributor

aschwith commented Aug 8, 2023

Ich habe die Tests gerade manuell nochmal angeschoben. Es gibt weiterhin Errors aus den automatisierten Tests. Ich lass den PR gerade auf meinem Produktivsystem laufen.

@psilo909
Copy link
Contributor

psilo909 commented Aug 8, 2023

habe es nun auch produktiv drinnen. wird aber etwas dauern, bis die session weg ist.

@psilo909
Copy link
Contributor

psilo909 commented Aug 9, 2023

Bisher keine Probleme. @aschwith bei Dir? Denke wir können mergen.

@aschwith
Copy link
Contributor

aschwith commented Aug 9, 2023

@psilo909 : Bei mir läuft es ohne Probleme. Wenn es bei Dir mit dem Erneuern der Session funktioniert, gerne mergen.

Copy link
Contributor

@aschwith aschwith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested ok with 7490

@psilo909
Copy link
Contributor

bis heute keine probleme mehr. ich merge

@psilo909 psilo909 merged commit 5e5c094 into smarthomeNG:develop Aug 10, 2023
5 checks passed
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.

4 participants