Minor issue with set command

[Mitul16]

In pwncat/commands/set.py

 12     def get_config_variables(self):
 13         options = ["state"] + list(self.manager.config.values)
 14
 15         if self.manager.target is not None:
 16             options.extend(user.name for user in self.manager.target.iter_users())
 17
 18         if self.manager.config.module:
 19             options.extend(self.manager.config.module.ARGUMENTS.keys())
 20
 21         return options

Why are we using ["state"] and adding usernames for get_config_variables?
The returned options are used to check for the validity of argument variable

Like set lhost 10.10.10.1 or set backdoor_user pwncat
Here, lhost and backdoor_user being the argument variable

The issue here is that after getting a session, get_config_variables returns a list containing "state" and usernames
This list can be checked by running set INVALID_ARG

Now, if we use one of the usernames or "state" as an argument variable like set root something, we get KeyError
Following is the reponsible code, in the same file - pwncat/commands/set.py

83                    manager.config.set(
84                        args.variable, args.value, getattr(args, "global")
85                    )

98                 value = manager.config[args.variable]
99                 console.print(
100                    f" [cyan]{args.variable}[/cyan] = [yellow]{repr(value)}[/yellow]"
101                )

Some minor checks can be added but why are we using "state" and usernames here?
["state"] was added back in 82ea579 here

---

There is one minor issue with sessions too
In pwncat/commands/sessions.py

56                 table.add_row(
57                     str(ident),
58                     session.current_user().name,
59                     str(session.hash),
60                     session.platform.name,
61                     str(type(session.platform.channel).__name__),
62                     str(session.platform.channel),
63                     **kwargs,
64                 )

Calling sessions.current_user() would fetch /etc/passwd and the whole command can fail if the session is not responding
For example, closing a network interface may cause this like closing TryHackMe openvpn while using pwncat