In SessionLimit::_onSessionCollision__DropOldest() there is a wrong property name being used: $session->id instead of $session->sid. A patch is coming,

CommentFileSizeAuthor
#2 2906310-2.session_limit.Wrong-property-name-in-onSessionCollisionDropOldest.patch830 bytesjurgenhaas
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jurgenhaas created an issue. See original summary.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
CreditAttribution: jurgenhaas at LakeDrops, PARAGON Executive Services GmbH commented
Status: Active » Needs review
FileSize
2906310-2.session_limit.Wrong-property-name-in-onSessionCollisionDropOldest.patch830 bytes
darksnow’s picture

darksnow CreditAttribution: darksnow at Deeson for Deeson commented
Status: Needs review » Closed (outdated)

Thanks for the patch jurgenhaas

The code in question now gets the session ID from the passed in $event so this patch is no longer needed. I'm closing this issue as out of date.