Problem/Motivation

This module causes masqueraded sessions to redirect on logout, which is not desired for that case.

Steps to reproduce

Install this masquerade & redirect_after_logout modules and configure both. When masquerading as a user, you will get sent to the page specified by redirect_after_logout instead of the users expected destination.

Proposed resolution

Check if the session has the 'masquerading' flag and bail, rather than redirect.

Remaining tasks

Decide if this is the best/nicest way of doing it.

CommentFileSizeAuthor
#3 3229727_dont_redirect_if_masquerading.patch667 bytessingularo
#2 dont_redirect_if_masquerading.patch667 bytessingularo

Comments

singularo created an issue. See original summary.

singularo’s picture

singularo
Location Adelaide, AUS
commented
StatusFileSize
new dont_redirect_if_masquerading.patch667 bytes

Patch attached that checks if masquerading before redirecting.

singularo’s picture

singularo
Location Adelaide, AUS
commented
StatusFileSize
new 3229727_dont_redirect_if_masquerading.patch667 bytes

Uploading better named patch file for future discovery.

  • 0cd5e09 committed on 8.x-1.x 
    Issue #3229727 by singularo, nevergone: Masquerade is interfered with by...
nevergone’s picture

nevergone
Primary language Hungarian
commented

Thanks the fix, test added.

nevergone’s picture

nevergone
Primary language Hungarian
commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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