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

ycom 3.0-beta4: logout funktioniert nicht #250

Closed
Goldfischpeter opened this issue Mar 13, 2019 · 10 comments
Closed

ycom 3.0-beta4: logout funktioniert nicht #250

Goldfischpeter opened this issue Mar 13, 2019 · 10 comments
Milestone

Comments

@Goldfischpeter
Copy link
Contributor

Logout wie in Doku beschrieben, loggt den User nicht aus. User bleibt eingeloggt.

@dergel dergel added this to the Version 3.0 milestone Mar 13, 2019
@Pixeldaniel
Copy link
Contributor

Manchmal wirkt der Logout-Prozess, als würde er sich aufhängen. Die Seite reagiert dann für Minuten nicht. Man wird manchmal auch aus dem REDAXO Backend geschmissen. Es macht einen Unterschied dabei, ob der Cache von REDAXO schon generiert war oder noch nicht. Ansonsten vollkommen willkürlich.

@alxndr-w
Copy link
Collaborator

@Pixeldaniel könnte ein lokales Problem sein. Bitte Bescheid geben, wenn du es mal getestet hast.

@IngoWinter
Copy link
Contributor

Bin gerade drüber gestolpert. Umgebung Windows/PHP7.2
Beim Logout wird hier: https://github.com/yakamara/redaxo_ycom/blob/master/plugins/auth/lib/ycom_auth.php#L402 der ycom Cookie geplättet. Allerdings ist $sessionConfig['frontend']['cookie']['path'] bei mir leer, der Cookie hat aber als Pfad /login/. Der Cookie wird wegen des unterschiedlichen Pfades nicht gelöscht, der User ist weiterhin eingeloggt. Eine Lösung habe ich ob der späten Stunde in Kombination mit einer nicht unerheblichen Menge an Weissweinschorle nicht zu bieten - das Problem habe ich mit setcookie('rex_ycom', null, -1, '/login/'); umgangen :)

@alxndr-w
Copy link
Collaborator

Das nächste Mal mit Rotwein? @dergel kannst du da was fixen nach so einer schönen Fehlerdiagnose von @IngoWinter ?

@hillepeter
Copy link

Hallo zusammen,

wir es für dieses Problem noch einen Fix geben?

Gruß
Daniel

@dergel
Copy link
Member

dergel commented Sep 16, 2019

das sind zwei Probleme gleichzeitig. Das mit dem Cookie in einem Unterordner kann man am besten lösen, und sollte man auch so machen, indem man in der redaxo config den cookie path auf '/' setzt. Das es manchmal so aussieht als ob alles hängt liegt an

rex_response::setHeader('Clear-Site-Data', '"cache", "cookies", "storage", "executionContexts"');

Diese Zeile löscht den kompletten Cache des Browser der Webseite. Und das ist dann manchmal schon einiges, so dass es einfach dauert, bis der Browser durch ist. Wird aber in der nächsten Version nicht mehr so sein.

https://github.com/yakamara/redaxo_ycom/blob/d33f4f42375ab370782384bb9672720adc152ed1/plugins/auth/lib/ycom_auth.php#L381

@IngoWinter
Copy link
Contributor

Also Redaxo setzt keinen Standard Pfad für Cookies, YCom ist aber auf einen Standard Pfad angewiesen, weil die Cookies sonst mit dem Pfad des Login Formulares gesetzt werden und dann der Logout evt. nicht funktioniert. Imho sollte dieses Verhalten prominent in der YCom Readme dokumentiert sein oder Redaxo sollte per Default / als Pfad setzen oder YCom sollte per Default / als Pfad nutzen. Wobei ich einen Standardpfad in der Redaxo Config vorziehen würde.

@alxndr-w
Copy link
Collaborator

Und wie mache ich das mit dem Standard-Pfad?

@IngoWinter
Copy link
Contributor

Ich denke mal hier (bzw. in unter /redaxo/data/core/config.yml): https://github.com/redaxo/redaxo/blob/master/redaxo/src/core/default.config.yml#L28

@dergel dergel modified the milestones: Version 3.0, Version 3.1 Sep 16, 2019
dergel added a commit that referenced this issue Sep 16, 2019
@dergel
Copy link
Member

dergel commented Sep 16, 2019

Cookie Pfad ist nun immer automatisch auf "/" gesetzt, überschreibt quasi den REDAXO Frontend Cookie Pfad.
Weiterhin ist sowieso einiges umgebaut, so dass auch 'Clear-Site-Data' nicht mehr genutzt wird, sondern die YCom sessionvariablen direkt gelöscht werden.

Bitte nicht die aktuelle github Version nutzen. Es ist noch einiges mehr umgebaut worde, was erst noch dokumentiert werden muss.

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

No branches or pull requests

6 participants