hi,
drupal 11.4.4
php 8.3.6

in user session page display The website encountered an unexpected error. Try again later.

in dblog page display php error;

TypeError: Drupal\session_inspector\SessionInspector::isMasqueradeSession(): Argument #1 ($data) must be of type array, true given, called in /home/......../public_html/web/modules/contrib/session_inspector/src/SessionInspector.php on line 80 in Drupal\session_inspector\SessionInspector->isMasqueradeSession() (line 135 of /home/......./public_html/web/modules/contrib/session_inspector/src/SessionInspector.php).

CommentFileSizeAuthor
#4 3612613-typeerror-isMasqueradeSession.patch607 bytesdkmishra

Issue fork session_inspector-3612613

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

bolaghi created an issue. See original summary.

dkmishra made their first commit to this issue’s fork.

dkmishra opened merge request !6

dkmishra’s picture

dkmishra commented
Version: 1.0.6 » 1.0.x-dev
Status: Active » Needs review
StatusFileSize
new 3612613-typeerror-isMasqueradeSession.patch607 bytes

I have created MR and also attached a patch. Please try it.

bolaghi’s picture

bolaghi commented

#4 work form me and solved problem.
thx

  • fc494987 committed on 1.0.x 
    Issue #3612613: Fixed session issue
philipnorton42’s picture

philipnorton42 commented
Status: Needs review » Fixed

Yup! Happy with this. Sorry about the late response, I spotted the issue on Tuesday but haven't been able to get to it until now as I've been too busy.

It's actually a little tricky to replicate, but some modules will add boolean values to the session data that causes this module to then crash. This solves that problem nicely.

Thanks @dkmishra for the fix! Very much appreciated! :)

MR has been merged. I'll tag a new release in a little bit.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

philipnorton42’s picture

philipnorton42 commented

Added to release 1.0.7.

https://www.drupal.org/project/session_inspector/releases/1.0.7

dkmishra’s picture

dkmishra commented

Hi @philipnorton42, thanks for committing the fix.

I noticed I wasn't credited for this issue. Would you mind adding issue credit for my contribution if you think it's appropriate?

Thank you!

philipnorton42’s picture

philipnorton42 commented

Sorry! Of course it's appropriate. I've updated the contribution list now.

philipnorton42’s picture

philipnorton42 commented
Status: Fixed » Closed (fixed)