Tibber: resubscribe on clean disconnect

[GrimmiMeloni]

fix #17925

Resubscribes in case of a clean exit.

[GrimmiMeloni]

@andig I refactored the existing subscription logic into a function (subscribe), and added an optional unsubscribe to it (in case we have a subscriptionId from a previous subscribe).

For the reconnect loop, I am using the presence of an error to decide wether to just restart Run() or also additionally re-subscribe. Maybe we can also get rid of that decision by simply always resubscribing? WDYT?

[andig]

Ich habs nicht versucht im Detail zu verstehen, aber die Idee ist klar- bin gespannt...

[andig]

ping @GrimmiMeloni

[GrimmiMeloni]

Sorry. Gerade unpässlich 🤒

[andig]

Ohje- gute Besserung!

[andig]

Mir fällt grad noch auf:

	go func() {
		<-ctx.Done()
		if err := client.Close(); err != nil {
			log.ERROR.Println(err)
		}
	}()

das ist vmtl. kontraproduktiv mit den Run darunter ;) Zumindest wird dort ja auch schon das Logging mit erledigt.

[GrimmiMeloni]

das ist vmtl. kontraproduktiv mit den Run darunter ;) Zumindest wird dort ja auch schon das Logging mit erledigt.

Was genau meinst Du wird kontraproduktiv sein? Für den Client ist es zumindest OK wenn dieser hier via Close() parallel geschlossen wird. Der stellt im schlimmsten Fall dann fest, das Ihm der parent Context zwichenzeitlich zugemacht wurde und beendet sich dann direkt wieder.

Oder siehst Du hier noch ein anderes Problem? Den Hinweis mit dem Logging verstehe ich nicht.