Jun 23, 2024 · Jun 23, 2024
diff --git a/CHANGELOG.md b/CHANGELOG.md
 ### Fixed
 - Fixed `shlex.join` use with non-str type objects (e.g. `RemotePath`)
 - Fixed `set` command use with incorrect keys (e.g. `set invalid value`)
 - Fixed `sessions` command for Windows platform with more robust variable type checks (e.g. `TypeError`)

 ### Added
 - Added missed `PlatformError` for `upload` command (e.g. "no gtfobins writers available")
diff --git a/pwncat/modules/windows/enumerate/user/__init__.py b/pwncat/modules/windows/enumerate/user/__init__.py
        except PowershellError as exc:
            raise ModuleFailed(str(exc)) from exc

        users = users[0]
 #users = users[0] # This enforces to only display the first session in list

        for user in users:
        if type(users) == list: # Extra check to prevent accidentally enumerating a dict as if it is a list
            for user in users:
                yield WindowsUser(
                    source=self.name,
                    name=user["Name"],
                    uid=user["SID"],
                    account_expires=None,
                    description=user["Description"],
                    enabled=user["Enabled"],
                    full_name=user["FullName"],
                    password_changeable_date=None,
                    password_expires=None,
                    user_may_change_password=user["UserMayChangePassword"],
                    password_required=user["PasswordRequired"],
                    password_last_set=None,
                    last_logon=None,
                    principal_source=user["PrincipalSource"],
                )
        elif type(users) == dict: # Extra check to make sure users is actually a dict
            yield WindowsUser(
                source=self.name,
                name=user["Name"],
                uid=user["SID"],
                account_expires=None,
                description=user["Description"],
                enabled=user["Enabled"],
                full_name=user["FullName"],
                password_changeable_date=None,
                password_expires=None,
                user_may_change_password=user["UserMayChangePassword"],
                password_required=user["PasswordRequired"],
                password_last_set=None,
                last_logon=None,
                principal_source=user["PrincipalSource"],
            )
                    source=self.name,
                    name=user["Name"],
                    uid=user["SID"],
                    account_expires=None,
                    description=user["Description"],
                    enabled=user["Enabled"],
                    full_name=user["FullName"],
                    password_changeable_date=None,
                    password_expires=None,
                    user_may_change_password=user["UserMayChangePassword"],
                    password_required=user["PasswordRequired"],
                    password_last_set=None,
                    last_logon=None,
                    principal_source=user["PrincipalSource"],
                )
        else:
            raise ModuleFailed("users from Get-Localuser is not a dict of one user, or a list of users!")

        well_known = {
            "S-1-0-0": "NULL AUTHORITY\\NOBODY",
Original file line number	Diff line number	Diff line change
Expand Up		@@ -11,6 +11,7 @@ and simply didn't have the time to go back and retroactively create one.
		### Fixed
		- Fixed `shlex.join` use with non-str type objects (e.g. `RemotePath`)
		- Fixed `set` command use with incorrect keys (e.g. `set invalid value`)
		- Fixed `sessions` command for Windows platform with more robust variable type checks (e.g. `TypeError`)

		### Added
		- Added missed `PlatformError` for `upload` command (e.g. "no gtfobins writers available")
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -23,25 +23,45 @@ def enumerate(self, session: "pwncat.manager.Session"):
		except PowershellError as exc:
		raise ModuleFailed(str(exc)) from exc

		users = users[0]
		#users = users[0] # This enforces to only display the first session in list

		for user in users:
		if type(users) == list: # Extra check to prevent accidentally enumerating a dict as if it is a list
		for user in users:
		yield WindowsUser(
		source=self.name,
		name=user["Name"],
		uid=user["SID"],
		account_expires=None,
		description=user["Description"],
		enabled=user["Enabled"],
		full_name=user["FullName"],
		password_changeable_date=None,
		password_expires=None,
		user_may_change_password=user["UserMayChangePassword"],
		password_required=user["PasswordRequired"],
		password_last_set=None,
		last_logon=None,
		principal_source=user["PrincipalSource"],
		)
		elif type(users) == dict: # Extra check to make sure users is actually a dict
		yield WindowsUser(
		source=self.name,
		name=user["Name"],
		uid=user["SID"],
		account_expires=None,
		description=user["Description"],
		enabled=user["Enabled"],
		full_name=user["FullName"],
		password_changeable_date=None,
		password_expires=None,
		user_may_change_password=user["UserMayChangePassword"],
		password_required=user["PasswordRequired"],
		password_last_set=None,
		last_logon=None,
		principal_source=user["PrincipalSource"],
		)
		source=self.name,
		name=user["Name"],
		uid=user["SID"],
		account_expires=None,
		description=user["Description"],
		enabled=user["Enabled"],
		full_name=user["FullName"],
		password_changeable_date=None,
		password_expires=None,
		user_may_change_password=user["UserMayChangePassword"],
		password_required=user["PasswordRequired"],
		password_last_set=None,
		last_logon=None,
		principal_source=user["PrincipalSource"],
		)
		else:
		raise ModuleFailed("users from Get-Localuser is not a dict of one user, or a list of users!")

		well_known = {
		"S-1-0-0": "NULL AUTHORITY\\NOBODY",
Expand Down