splunk / contentctl

Splunk Content Control Tool
Apache License 2.0
80 stars 20 forks source link

Fix checking during mode:changes #187

Closed pyth0n1c closed 2 months ago

pyth0n1c commented 2 months ago

Fix bug where a detection that changed would not be tested if it contained the string "dist" in the name. This was due to how we checked whether or not modified content was in the appropriate content folders like detections, macros, and lookups or whether it was in a folder like dist/ or app_template. The way this checking is done is now improved so we examine the paths using is_relative_to instead of looking for a substring in the path.

patel-bhavin commented 2 months ago

Full path checking, FTW!