Skip to content

Fix non-authenticated users being counted as sessions#362

Merged
DecDuck merged 1 commit intoDrop-OSS:developfrom
Pacodastre:fix-inactive-users-on-dashboard
Mar 2, 2026
Merged

Fix non-authenticated users being counted as sessions#362
DecDuck merged 1 commit intoDrop-OSS:developfrom
Pacodastre:fix-inactive-users-on-dashboard

Conversation

@Pacodastre
Copy link
Contributor

…dashboard

@DecDuck
Copy link
Member

DecDuck commented Mar 1, 2026

Does this do anything? userId can't be null.

@Pacodastre
Copy link
Contributor Author

Not entirely sure why, but the following request

        await prisma.session.groupBy({
          by: ["userId"],
        }),

returns this

[ { userId: null }, { userId: '46d4ebaf-e79d-427e-b3ba-2162dafaeb9a' } ]

I'm not sure why it's not "system".

@DecDuck
Copy link
Member

DecDuck commented Mar 2, 2026

Can you confirm that's the system user? Does it have it's username or anything?

@DecDuck DecDuck changed the title Fixes a bug where the amount of inactive users is wrong on the admin … Fix non-authenticated users being counted as sessions Mar 2, 2026
@DecDuck DecDuck merged commit 3a9c61a into Drop-OSS:develop Mar 2, 2026
2 checks passed
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