Currently Masquerade relies on the fact that sessions are stored in the database, which is not always true. Let's try to have something flexible and add support for the Memcache Storage module.

CommentFileSizeAuthor
#6 2753469-masquerade_memcache_storage-3.patch2.06 KBAron Novak
#5 2753469-masquerade_memcache_storage-2.patch2.21 KBAron Novak
#2 2753469-masquerade_memcache_storage-1.patch2.21 KBAron Novak
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Aron Novak created an issue. See original summary.

Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak at Agence Inovae commented
FileSize
2753469-masquerade_memcache_storage-1.patch2.21 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2753469-masquerade_memcache_storage-1.patch, failed testing.

The last submitted patch, 2: 2753469-masquerade_memcache_storage-1.patch, failed testing.

Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak at Agence Inovae commented
FileSize
2753469-masquerade_memcache_storage-2.patch2.21 KB
Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak at Agence Inovae commented
FileSize
2753469-masquerade_memcache_storage-3.patch2.06 KB
Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak at Agence Inovae commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 6: 2753469-masquerade_memcache_storage-3.patch, failed testing.

The last submitted patch, 6: 2753469-masquerade_memcache_storage-3.patch, failed testing.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Related issues: +#1813696: Set/unset Session flag when masquerade status changes.

I think we should not work with session table(storage directly)

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented

In 8.x branch masquerade is flag in session

mvc’s picture

mvc
they
Primary language English
Location Montréal, CA
CreditAttribution: mvc at McGill University commented

#1926074: Remove {masquerade} table and rely on session flag only will also resolve this.

mvc’s picture

mvc
they
Primary language English
Location Montréal, CA
CreditAttribution: mvc at McGill University commented
Related issues: +#1926074: Remove {masquerade} table and rely on session flag only