DesktopCommanderMCP icon indicating copy to clipboard operation
DesktopCommanderMCP copied to clipboard

set_config seems pretty dangerous

Open genesiscz opened this issue 6 months ago • 2 comments

I mean, wouldn't it be better to just use the configuration params while starting the server, instead of making sure the model is not just calling whatever it wants? This seems like a pretty serious security concern and I think it would be useful to just don't let the model change the configuration insted of having this warning.

genesiscz avatar May 05 '25 10:05 genesiscz

Its a balance/compromise for non technical users. Most people will not edit startup config. So we wanted chatbot to be able to help them to configure.

Since we wrote that warning things also changed in Claude, now you can disable tools one by one

Image

So you have option to remove that + edit config json by hand if you want

wonderwhy-er avatar May 07 '25 08:05 wonderwhy-er

I do think it can be done better but we are trying to find best compromise given we do not have control over Claude client ui/ux

wonderwhy-er avatar May 07 '25 08:05 wonderwhy-er