Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PT Run] Disabling shell plugins does not work #10472

Open
cory-chang opened this issue Mar 28, 2021 · 4 comments
Open

[PT Run] Disabling shell plugins does not work #10472

cory-chang opened this issue Mar 28, 2021 · 4 comments
Labels
Issue-Bug Something isn't working Priority-3 Bug that is low priority Run-Plugin Things that relate with PowerToys Run's plugin interface

Comments

@cory-chang
Copy link

Microsoft PowerToys version

0.33.1

Running as admin

Yes

Area(s) with issue?

PowerToys Run

Other Software

No response

Steps to reproduce

  1. Turn off "Shell" under Plugins in PT Run
  2. Type in a command, such as "ping"
  3. See results for the executing the command ping

✔️ Expected Behavior

Nothing to show up, because I have disabled executing shell commands from PT Run

❌ Actual Behavior

I see commands show up, see screenshot below

image

@cory-chang cory-chang added Issue-Bug Something isn't working Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Mar 28, 2021
@enricogior enricogior added Run-Plugin Things that relate with PowerToys Run's plugin interface and removed Issue-Bug Something isn't working Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Mar 28, 2021
@enricogior
Copy link
Contributor

@ccory2004
this is expected, the result is returned by the program plugin.

@cory-chang
Copy link
Author

@enricogior Hm, I guess that makes sense, but I think the disabling shell commands should really disable anything run in the shell, like ping. Do you know if there is a way to differentiate between gui programs and cli programs?

@htcfreek
Copy link
Collaborator

@ccory2004

Hm, I guess that makes sense, but I think the disabling shell commands should really disable anything run in the shell, like ping.

What is the use case/reason for this?

@cory-chang
Copy link
Author

cory-chang commented Mar 28, 2021

@htcfreek Its mostly just stylistic, I use PT Run mainly for apps that I use on a daily basis and files, and the commands get in the way of it sometimes.

But I do think it is a bit misleading (if understandable) that disabling shell doesn't actually disable most commands, which makes the option not super useful. Maybe part of it has to do with the examples that are given in the description (ping and cmd), both of which still appear even after disabling them, as well as the wording underneath them still reads "Run command", which really references the idea of executing a shell command, as opposed to applications that show "Application" below them.

image

Also, forgive my ignorance, but I wasn't able to come up with a single command that no longer works with the Shell plugin turned off, but maybe I'm missing something.

Sorry for the long-ish response.

@cinnamon-msft cinnamon-msft added Issue-Bug Something isn't working Priority-3 Bug that is low priority labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug Something isn't working Priority-3 Bug that is low priority Run-Plugin Things that relate with PowerToys Run's plugin interface
Projects
Status: No status
Development

No branches or pull requests

4 participants