Skip to content

Improve efficiency of consoleusers.CurrentUids and consoleusers.ExplorerProcess #2463

@RebeccaMahany

Description

@RebeccaMahany

Right now, on Windows, the new consoleusers.CurrentUids implementation added in #2422 is incorrectly identifying additional users that we should start desktop processes for. This means that the desktop runner is, every five seconds, trying to start a desktop process for these users, which means it calls consoleusers.ExplorerProcess and enumerates every single process. This is very noisy in the logs, and we suspect there's a performance impact. (This issue will, though, be invisible to the end user otherwise.)

For the next release:

Over the next two releases:

In the long term:

  • Investigate subscribing to logon events so that we don't have to run this check quite so frequently (or maybe we can phase it out altogether)

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions