Skip to content

fix(midea): MSmartHome login numeric types (value is illegal) - #198

Merged
CallMeTechie merged 1 commit into
masterfrom
fix/midea-msmarthome-login
Jun 26, 2026
Merged

fix(midea): MSmartHome login numeric types (value is illegal)#198
CallMeTechie merged 1 commit into
masterfrom
fix/midea-msmarthome-login

Conversation

@CallMeTechie

Copy link
Copy Markdown
Owner

Midea: MSmartHome-Cloud-Login „value is illegal" beheben

Befund (real, gemeldet): MSmartHome-Login wird von der Midea-Cloud mit „value is illegal" abgelehnt.

Ursache (byte-genauer Abgleich gegen msmart-ng/cloud.py): Wir sendeten format/clientType (Common-Request-Body) und platform/clientType (Login-data/iotData) als Strings ("2"/"1"). Die Referenz nutzt Zahlen (BaseCloud.FORMAT = 2, CLIENT_TYPE = 1). Die Midea-Cloud validiert diese Felder typstreng und antwortet bei "2"/"1" mit „value is illegal".

Fix:

  • format: 2, clientType: 1 (numerisch) im MSmartHome-Common-Body.
  • data.platform: 2, iotData.clientType: 1 (numerisch) im /mj/user/login-Body.
  • Fehlermeldungen führen jetzt den Midea-Fehlercode mit (… (Midea-Code <n>)), damit ein etwaiger Rest-Fehler ohne Live-Zugang weiter diagnostiziert werden kann.

Tests: Neuer deterministischer Test stubt fetch und prüft, dass der MSmartHome-Request format/clientType als number (nicht string) sendet, inkl. Common-Felder + Passthrough. Volle midea-Suite grün (42 Tests, 40 pass / 2 Live-Skip / 0 fail).

Wichtig — Live-Verifikation steht aus: Der eigentliche Cloud-Login lässt sich nur gegen die echte Midea-Cloud beweisen (kein CI-Zugang). Wenn nach Deploy weiterhin ein Fehler kommt, zeigt die Meldung jetzt den Midea-Code → nächster gezielter Schritt. Andernfalls bleibt NetHome Plus der dokumentierte Fallback.

🤖 Generated with Claude Code

@CallMeTechie
CallMeTechie merged commit e977c9a into master Jun 26, 2026
8 checks passed
@CallMeTechie
CallMeTechie deleted the fix/midea-msmarthome-login branch June 26, 2026 20:36
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.

1 participant