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

Cannot add integration with 2FA #156

Open
saschabrockel opened this issue Dec 29, 2023 · 16 comments · Fixed by #160
Open

Cannot add integration with 2FA #156

saschabrockel opened this issue Dec 29, 2023 · 16 comments · Fixed by #160
Labels
wait in examination

Comments

@saschabrockel
Copy link

saschabrockel commented Dec 29, 2023

The manual is not really clear for the last step. I did everything and know have to type in my credentials and login URL and 2FA. But what should I paste in 2FA? No matter what I try, it doesn't work.

It always returns: Unerwarteter Fehler. Unexpected error in English.

@Ludy87
Copy link
Owner

Ludy87 commented Dec 29, 2023

@saschabrockel
Copy link
Author

@Ludy87 ja. Was muss ich denn bei Zwei Faktor Authentifizierung eintragen? Den Code vom Sensor?

@saschabrockel
Copy link
Author

Jetzt gab es einmal den folgenden Fehler in den Logs:
('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

Seitdem wieder nur Unerwarteter Fehler als Popup und nichts in den Logs.

@Ludy87
Copy link
Owner

Ludy87 commented Jan 3, 2024

Ista ist da etwas komisch im Verhalten, du muss die Integration neu laden.

image

@saschabrockel
Copy link
Author

Wie soll ich denn neu laden, wenn ich noch keinen Konfigurationseintrag habe?

@Ludy87
Copy link
Owner

Ludy87 commented Jan 3, 2024

Bin davon ausgegangen, durch die Fehlermeldung: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

Okay, dann nochmal von vorne.

Hast du den Sensor angelegt:

# Example configuration.yaml entry
sensor:
  - platform: otp
    token: KEY_FROM_ECOTREND

KEY_FROM_ECOTREND wird hier erklärt: https://github.com/Ludy87/ecotrend-ista/blob/main/TWO_FACTOR_AUTHENTICATION_DE.md#klicke-auf-hat-der-scan-nicht-funktioniert-unter-dem-qr-code

@saschabrockel
Copy link
Author

saschabrockel commented Jan 3, 2024

Ja, habe ich. Habe den auch in der Oberfläche:
image

Die Meldung kam leider auch nur einmal. Sonst eben immer der unerwartete Fehler ohne Logs. Schwer so zu debuggen.

@saschabrockel
Copy link
Author

Du meinst, dass ich den Code in Home Assistant durch den Neustart habe und dann auf der Ista Seite quasi schonmal den Namen vorausgefüllt habe und dann nur noch den Code aus Home Assistant auf die Ista Seite kopiert habe und dann quasi abgeschickt. Und das eben innerhalb der 30 Sekunden Gültigkeit oder?

Das habe ich definitiv gemacht. Ich glaube sonst wäre er ja auch nicht hier oder:
image

Ist es ein Problem, da ich zwei Stück habe?

@saschabrockel
Copy link
Author

@Ludy87 soll der Daumen hoch heißen, dass es ein Problem ist? 😄

@saschabrockel
Copy link
Author

Will nicht nerven, aber gerade die letzte Antwort zu wissen wäre noch gut 😄

@Ludy87
Copy link
Owner

Ludy87 commented Jan 6, 2024

Ich weiß momentan was das Problem ist. Aber es sind erstmal alle Schritte getan die getan werden müssen.

@Ludy87 Ludy87 mentioned this issue Jan 14, 2024
@saschabrockel
Copy link
Author

Hat leider nichts gebracht. Hab geupdated, HA neugestartet und es geht nach wie vor nicht.

@Ludy87 Ludy87 reopened this Jan 15, 2024
@Ludy87
Copy link
Owner

Ludy87 commented Jan 15, 2024

Sorry das es nicht funktioniert, bin etwas verzweifelt. Leider kann ich es auch nicht erneut durchspielen.

@Ludy87 Ludy87 added the wait in examination label Jan 15, 2024
@saschabrockel
Copy link
Author

Ich kann gerne versuchen zu helfen, soweit ich kann. Also jedenfalls wenn es um das Testen von Pre-Release Versionen geht etc.

@habandy
Copy link

habandy commented Apr 14, 2024

@Ludy87
Ich hab genau das gleiche Problem.
Ersteinrichtung mit HACS auf HAOS 12.2, Core Version 2024.4.3.

Ich habe die Anleitung befolgt und den Schlüssel vom "Scan hat nicht funktioniert" - Link in die OTP Entität kopiert. Gleichzeitig habe ich den Schlüssel aber auch verwendet um den Code auch auf dem Handy im Google Authenticator zu geniereren. Da beides den gleichen Schlüssel verwenden sollte das aber kein Problem sein.

Beim Versuch die Integration hinzuzufügen erscheint wie von @saschabrockel beschrieben nach dem Einfügen des OTP-Codes und dem Klick auf den Button "Absenden" nach einigen Sekunden die Fehlermeldung "es ist ein unerwarteter Fehler aufgetreten."

Im Log findet man dann folgende Einträge:

Logger: urllib3.connectionpool
Quelle: /usr/local/lib/python3.12/site-packages/urllib3/connectionpool.py:824
Erstmals aufgetreten: 09:53:55 (1 Vorkommnisse)
Zuletzt protokolliert: 09:53:55

Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ReadTimeoutError("HTTPSConnectionPool(host='keycloak.ista.com', port=443): Read timed out. (read timeout=10)")': /realms/eed-prod/protocol/openid-connect/auth?response_mode=fragment&response_type=code&client_id=ecotrend&scope=openid&redirect_uri=https%3A%2F%2Fecotrend.ista.de%2Flogin-redirect&code_challenge=W7-_bzEvPJ78TAH9pxOzPvQV558q22iEKz-YQ4Zjtx8&code_challenge_method=S256

Bezüglich der Anleitung:
Die Anleitung ist vielleicht dahingehend ein wenig verwirrend geschrieben weil in der Beschreibung schnell unklar ist welcher Schlüssel, Key, Token oder Code gemeint ist, wo dieser hin muss und welchen man jetzt wo eintragen muss.
Leute die mit solchen Begriffen nicht umgehen können haben es zusätzlich schwerer.

Ich empfinde jedenfalls den Punkt "erstell nun einen OTP Sensor in Home-Assistant und startet Home-Assistant neu" nur mit blanken Code-Beispiel als nicht ausreichend beschrieben / erklärt.
Und beim nachfolgenden Punkt die Bemerkung mit der 30-Sekunden Gültigkeit verwirrend weil der dazugehörige Screenshot noch zur Geräteeinrichtung auf der ISTA-Webseite gehört, der OTP-Code aber nur relevant beim Einrichten der Integration ist.

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

Successfully merging a pull request may close this issue.

3 participants