Skip to content

Conversation

@cshagen
Copy link
Collaborator

@cshagen cshagen commented Jan 1, 2025

  • Korrektur Anzeige der SOCs im linken Graphen
  • Wechsel des Cookie-Modes (Samesite) von "Strict" zu "None"
  • Limitierung der Namenlänge im alternative Energie-Widget
  • Versions-Aktualisierung von Node-Dependencies
  • Kleinere Bug-Korrekturen

@benderl benderl added bug Something isn't working enhancement New feature or request dependencies Pull requests that update a dependency file labels Jan 2, 2025
@benderl benderl self-assigned this Jan 2, 2025
'openWBColorTheme=' +
JSON.stringify(prefs) +
';max-age=16000000;samesite=strict'
';max-age=16000000;samesite=none'
Copy link
Contributor

Choose a reason for hiding this comment

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

Ich denke nicht, dass diese Einstellung ausreichend ist.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie
Es fehlt die Angabe "Secure". Das setzt jedoch wieder voraus, dass die Verbindung über https erfolgt. Über http können dann keine Cookies mehr gesetzt werden, da "SameSite=None" von modernen Browsern dann blockiert wird.

Ich würde überlegen, ob es nicht langsam an der Zeit ist, die Theme-Optionen in die globalen Einstellungen zu verlegen. Das hat zwar zur Folge, dass diese Einstellungen für alle Endgeräte gelten, wäre aber dann konform zu den anderen Themes von openWB. Wenn später Benutzerprofile verfügbar sind, kann wieder jeder Anwender "seine" Einstellungen setzen.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Danke für den Hinweis!
Ich denke halt nicht, dass ein einzelner Satz von Optionen über alle Endgeräte hinweg sinnvoll ist. Auf dem Handy schalte ich manche Elemente vielleicht gerne aus, die auf dem grossen Bildschirm sinnvoll sind. Da helfen die Cookies schon.
Möglicherweise braucht es einen Split zwischen generellen Präferenzen, die für alle Endgeräte gesetzt sind (z.B. Anzahl Nachkommastellen oder Darstellung des Powergraphen) und Ein/Ausschalten bestimmter Features (z.B. Uhrzeit in der Kopfzeile). Ersteres würde auf dem Server gespeichert, letzteres im Cookie.
Einstweilen schalte ich mal "Secure" ein. Die openwb scheint ja https zu unterstützen, wenn man das self-signed Zertifikat akzeptiert.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ich habe das jetzt konfigurierter gemacht. Standard-Einstellung ist wie bisher "Samesite=Strict". In den Theme-Einstellungen lässt sich das auf "SameSite=None;Secure" ändern. Damit können die Iframe-Nutzer experimentieren.

@cshagen cshagen requested a review from benderl January 2, 2025 09:59
…ngs.vue

Co-authored-by: benderl <benderl@users.noreply.github.com>
@cshagen cshagen requested a review from benderl January 2, 2025 14:10
@benderl benderl merged commit d98adec into openWB:master Jan 6, 2025
1 check passed
@benderl benderl removed their request for review January 6, 2025 07:00
@cshagen cshagen deleted the update-web branch February 9, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants