Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[3.5.6] Funktion Benutzer wechseln ändert alle Sessions des ursprünglichen Benutzers #8158

Closed
Mynyx opened this issue Dec 10, 2015 · 10 comments
Closed

[3.5.6] Funktion Benutzer wechseln ändert alle Sessions des ursprünglichen Benutzers #8158

Mynyx opened this issue Dec 10, 2015 · 10 comments
Assignees
@leofeyer
Labels
defect
Milestone
3.5.10

Comments

@Mynyx
Copy link
Contributor

Mynyx commented Dec 10, 2015

Die Funktion "Zu Benutzer wechseln" ändert alle Sessions des ursprünglichen Benutzers, statt nur die Session zu ändern, in welcher der ursprüngliche Benutzer die Funktion ausführt.

Siehe https://github.com/contao/core/blob/master/system/modules/core/dca/tl_user.php#L627
@Zeromax
Copy link

Zeromax commented Dec 10, 2015

Ah.. This is the reason why I am logged in as a diffent account in an other browser...

@leofeyer
Copy link
Member

?

@Mynyx
Copy link
Contributor Author

Mynyx commented Dec 15, 2015

Einloggen im Backend mit dem selben Benutzer in zwei verschiedenen Browsern. In einem Browser über die Benutzerliste zu einem anderen Backend Benutzer wechseln. Im anderen Browser ist man dann ebenfalls mit dem anderen Benutzer eingeloggt, obwohl es sich um verschiedene Sessions handelt.

@aschempp
Copy link
Member

Es handelt sich für Contao um dieselbe Session. Users sind persönlich und sollten nicht mehrfach benutzt werden.

@Mynyx
Copy link
Contributor Author

Mynyx commented Dec 15, 2015

In tl_session werden die Sessions separat geführt, insofern verstehe ich nicht was du meinst?

Außerdem geht es ja gerade darum, dass die selbe Person sich mehrfach anmeldet, und nicht um verschiedene handelnde Personen.

@Mynyx
Copy link
Contributor Author

Mynyx commented Mar 17, 2016

Fehlen hierzu noch weitere Informationen?

@leofeyer
Copy link
Member

@contao/developers /cc

@aschempp
Copy link
Member

Müssten wir ein neues Session- und Login-Cookie erzeugen?

@Zeromax
Copy link

Zeromax commented Apr 20, 2016

Ich denke es würde reichen den richtigen Session Eintrag aus der Tabelle tl_session zu löschen und nicht alle.

@leofeyer leofeyer added defect and removed feature labels Apr 20, 2016
@leofeyer leofeyer added this to the 3.5.10 milestone Apr 20, 2016
@leofeyer leofeyer self-assigned this Apr 20, 2016
leofeyer added a commit that referenced this issue Apr 20, 2016
@leofeyer
Do not change all sessions when switching users (see #8158).
f8da814
@leofeyer
Copy link
Member

Behoben in f8da814.

leofeyer added a commit that referenced this issue Apr 21, 2016
@leofeyer
Do not change all sessions when switching users (see #8158).
b6ce856
jsonn pushed a commit to jsonn/pkgsrc that referenced this issue Apr 23, 2016
Update contao41 to 4.1.3.
f8a409f 
### 4.1.3 (2016-04-22)

 * Use data URIs for the image preview in the back end.
 * Use DIRECTORY_SEPARATOR to convert kernel.cache_dir into a relative path (see #464).
 * Always trigger the "isVisibleElement" hook (see contao/core#8312).
 * Do not change all sessions when switching users (see contao/core#8158).
 * Do not allow to close fieldsets with empty required fields (see contao/core#8300).
 * Make the path related properties of the File class binary-safe (see contao/core#8295).
 * Correctly validate and decode IDNA e-mail addresses (see contao/core#8306).
 * Skip forward pages entirely in the book navigation module (see contao/core#5074).
 * Do not add the X-Priority header in the Email class (see contao/core#8298).
 * Determine the search index checksum in a more reliable way (see contao/core#7652).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@leofeyer leofeyer

Labels
defect
Projects
None yet
Milestone
3.5.10
Development

No branches or pull requests
4 participants
@aschempp @leofeyer @Mynyx @Zeromax