Skip to content

NOISS Check for auth in composer before checking impersonation#893

Merged
iccowan merged 1 commit intomainfrom
noiss-fix-impersonation-no-auth
Mar 30, 2026
Merged

NOISS Check for auth in composer before checking impersonation#893
iccowan merged 1 commit intomainfrom
noiss-fix-impersonation-no-auth

Conversation

@iccowan
Copy link
Copy Markdown
Member

@iccowan iccowan commented Mar 26, 2026

This PR will:

  • Fix an issue where non-logged in users are unable to access the website when impersonation is enabled
  • Allow enabling impersonation on prod (LEAVE DISABLED UNTIL THIS IS MERGED AND DEPLOYED)

Copy link
Copy Markdown
Contributor

@kjporter kjporter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA good - confirmed that this fixes the bug. Approved to merge.

@iccowan iccowan force-pushed the noiss-fix-impersonation-no-auth branch from e8f5ce8 to e1b08d8 Compare March 30, 2026 15:29
@iccowan iccowan merged commit e77318a into main Mar 30, 2026
13 checks passed
@iccowan iccowan deleted the noiss-fix-impersonation-no-auth branch March 30, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants