The core change #2228393: Decouple session from cookie based user authentication broke the masquerading from ever completing. A success message is displayed, but the user is not properly switched.

CommentFileSizeAuthor
#2 masquerade-sessions-fix-2552881-02.patch911 bytesjhedstrom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhedstrom created an issue. See original summary.

jhedstrom’s picture

Status: Active » Needs review
FileSize
911 bytes

Patch depends on #2543678: Call to undefined method Drupal\Core\Render\Renderer::generateCachePlaceholder().

Tests in MasqueradeTest go green with this and that patch. However, the access tests are still failing, and I haven't had a chance to look at those.

andypost’s picture

Status: Needs review » Reviewed & tested by the community
Parent issue: » #1836516: Port Masquerade to D8 and rewrite + simplify it into a new 2.x series

  • andypost committed 04fcbe9 on 8.x-2.x authored by jhedstrom
    Issue #2552881 by jhedstrom: Changes necessary from core decoupling of...
andypost’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.