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

Easee: reauthenticate when refresh fails #9720

Merged
merged 5 commits into from
Sep 5, 2023

Conversation

GrimmiMeloni
Copy link
Collaborator

fix #9717

charger/easee/identity.go Outdated Show resolved Hide resolved
charger/easee/identity.go Outdated Show resolved Hide resolved
@andig andig added the enhancement New feature or request label Sep 4, 2023
@andig andig merged commit d7ca1fc into evcc-io:master Sep 5, 2023
6 checks passed
@jheinitz
Copy link
Contributor

jheinitz commented Sep 5, 2023

Danke an euch zwei Helden! Wahrscheinlich passiert jetzt für 12 Monate nix an der Easee API, aber das ist dann auch egal.... Also, vielen Dank und Prost @GrimmiMeloni :-)

@allcoolusernamesaregone
Copy link
Contributor

Also ich dachte ich hätte schon gefragt, aber finde es nicht mehr wo bzw. weiß nun nicht mehr ob.
Aber ich würde ja gerne EVCC im Master mit den ganzen aktuellen Änderungen zur Easee (start statt resume, die Überarbeitung der Commits und dies hier (reauthenticate) "in Summe" testen wollen, aber im Master sind die Änderungen zum Authentifizieren und Ladestart über start_charging ja nicht drin.. gibt's da einen Zeitplan für?

@jheinitz
Copy link
Contributor

jheinitz commented Sep 7, 2023

Doch! Die letzten Änderungen für die Easee sind im master enthalten. Entweder nimmst Du das nightly Docker Image oder Du checkst den Master Branch aus und baust es lokal.

Hier gibt es weitere Infos dazu: https://github.com/evcc-io/evcc#contribute

Edit: Link hinzugefügt

@allcoolusernamesaregone
Copy link
Contributor

Der PR #9271 ist aber ja nicht im Master, der fix mit reauth aber schon.. daher ja mein Problem, bzw. fehlendes Verständnis wie ich den master mit sonstigen noch nicht im Master befindlichen PR Änderungen sauber zusammenkriege, bzw idealerweise einfach alles im Master ist...

@GrimmiMeloni
Copy link
Collaborator Author

@allcoolusernamesaregone reicht dir ein Docker image? Dann nimm grimmimeloni/evcc:nightly da sind sowohl reauth hier, und auch das start_charge feature drin.

@allcoolusernamesaregone
Copy link
Contributor

Also ich habe Docker schonmal gehört, aber bin wie ein Raucher sagen würde Selbstdreher, kompiliere selbst auf meinem SBC (was den genau schön schwitzen lässt 😂)

@allcoolusernamesaregone
Copy link
Contributor

Ist für mich jetzt nicht eilig das ganze, dachte nur es könnte ja schon sinnvoll sein die ganzen Änderungen nochmal in Gänze zu testen...

@GrimmiMeloni
Copy link
Collaborator Author

Na dann schnapp dir halt Master und merge bei dir lokal noch den Feature Branch für Start_charge mit drauf. Dann hast Du es.

@jheinitz
Copy link
Contributor

jheinitz commented Sep 7, 2023

Der PR #9271 ist aber ja nicht im Master, der fix mit reauth aber schon.. daher ja mein Problem, bzw. fehlendes Verständnis wie ich den master mit sonstigen noch nicht im Master befindlichen PR Änderungen sauber zusammenkriege, bzw idealerweise einfach alles im Master ist...

Ah okay, sorry. Mein Fehler. Selbst schon ganz durcheinander 😉

@allcoolusernamesaregone
Copy link
Contributor

Ah okay, sorry. Mein Fehler. Selbst schon ganz durcheinander 😉

kein Ding, nichts passiert :)

@allcoolusernamesaregone
Copy link
Contributor

Feature Branch für Start_charge mit drauf.

äh aber wo finde ich den? ich hab keine Ahnung wie das geht.. bisher habe ich die Änderungen aus dem PR von Hand im Dateisystem nachgezogen bzw Dateien drüber gebügelt. Aber so geht's jetzt ja nicht mehr..
ich glaub ich warte bis dass alles drin ist 😅

@dm82m
Copy link

dm82m commented Sep 9, 2023

@GrimmiMeloni sag mal: wenn du in deinem fork den nightly händisch laufen lässt, wie überschreibst du das docker Target repo? In deiner nightly steht ja evcc/evcc:nightly aber dein nightly published ja auf grimmi/…

ich würde das gerne wissen, damit ich auf Home Assistant dein easee auth branch testen kann. Danke für die Hilfe!

@GrimmiMeloni
Copy link
Collaborator Author

GrimmiMeloni commented Sep 9, 2023

@dm82m der Auth branch wurde gerade gemerged, also kannst Du Dir die Arbeit sparen. ;)

Ich erkläre trotzdem gerne meinen Workflow, vielleicht hilft es Dir (oder anderen) in Zukunft.

Ich habe mir einen Branch erzeugt, der die notwendigen Änderungen am nightly Job enthält.

  • Branch Name ist dynamisch (statt immer master)
  • kein APT publish
  • schreibt in mein Docker Repo

Ich trenne dann immer so auf, daß ich Änderungen für evcc in einem separaten Branch entwickle (das ist der, den ich am Ende auch als Pull Request für evcc anfrage). Zum Testen erzeuge ich mir dann einen zweiten Branch von Master, und in den merge ich meinen aktuellen Change und zusätzlich den oben genannten Branch mit den GitHub Actions Änderungen. Den resultierenden Branch kann ich nach einem Push dann über die GH Oberfläche bauen. Einfach Actions, Nightly build, und den passenden Branch auswählen. Nach 10 Minuten liegt dann das Container image im docker hub.

@dm82m
Copy link

dm82m commented Sep 9, 2023

mega, danke für die erklärung, hilft mir für die zukunft!

@GrimmiMeloni GrimmiMeloni deleted the feature/easee-reauth branch October 20, 2023 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Easee: reauthenticate on failing token refresh
5 participants