-
Notifications
You must be signed in to change notification settings - Fork 107
Colors theme (web): Fixes and version upgrades #2110
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
Conversation
| 'openWBColorTheme=' + | ||
| JSON.stringify(prefs) + | ||
| ';max-age=16000000;samesite=strict' | ||
| ';max-age=16000000;samesite=none' |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
packages/modules/web_themes/colors/source/src/views/ThemeSettings.vue
Outdated
Show resolved
Hide resolved
…ngs.vue Co-authored-by: benderl <benderl@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.