search

CrowdStrike / falcon-scripts

Scripts to streamline the deployment and use of the CrowdStrike Falcon sensor
The Unlicense
153 stars 89 forks source link

Fix issue with case inside if statement inside shell expansion #113

Closed carlosmmatos closed 1 year ago

carlosmmatos commented 1 year ago

Fixes #108

There is some weird issue when trying to have a case statement inside and if block, inside a shell expression $(). This keeps the same logic, just moves the if statements outside the expansion to allow the case to work as intended.

carlosmmatos commented 1 year ago

@falcon-pioupiou can you review this, as you introduced these changes, and I have no idea how they worked for you. I definitely want to keep the functionality, but this seems to be a safer way to accomplish what you intended.