DefGuard / gateway

Defguard gateway
Other
22 stars 3 forks source link

use shell to execute command #134

Closed laoshancun closed 1 week ago

laoshancun commented 1 week ago

📖 Description

Close #133

🛠️ Dev Branch Merge Checklist:

Documentation

Testing

Deployment

🏚️ Main Branch Merge Checklist:

Testing

Documentation

teon commented 1 week ago

@laoshancun first of all, thank you very much for your contribution and effort, but the one command in pre/post up/down is by design - as defguard is built with highest security standards in mind.

If someone requires multiple commands to be executed, they can create a shell script which will define the acceptable and preferred shell to use and then execute all commands.

Why?

Each administrator has a preferred shell, and also in the script they can do other arrangements to control the environment.

Also what you have proposed - to launch the "sh" shell is not secured, as it's first sh shell from the path - that can be used for example to exploit the server on the gateway. And for that reason, we require the administrator to create a script in which **the shell is defined by the admin as well as accepted by the admin (can be full path, from env - but the admin accepts what it is).

So for that reason we can't merge this. Thank you again and hope you will understand our motivation.