feat(panic): add panic handler to possible panic places

[pereiramarco011]

Proposed Changes

• Increase KICS scan resiliency
• Increasing scan resiliency by gracefully handling errors during scans #6521

I submit this contribution under the Apache-2.0 license.

[github-actions]

KICS version: v1.7.4

Category Results HIGH 0 MEDIUM 0 LOW 0 INFO 0 TRACE 0 TOTAL 0	Metric Values Files scanned 1 Files parsed 1 Files failed to scan 0 Total executed queries 49 Queries failed to execute 0 Execution time 1

[kaplanlior]

@liorj-orca, I would appreciate your feedback on this PR.
CC @lior-orca

[lior-orca]

@pereiramarco011 besides maybe having a common helper method to build and print the error, the change looks good and important.
I suggest pushing it forward, in order to increase the resiliency.
We can always add more support to other uncovered areas

[liorj-orca]

@pereiramarco011 overall looks good. As @lior-orca stated, I think we can push the fix as is and add more cases later on along the way.
One more case that you can think wether or not we want to avoid the error:

• Getsources function (

  kics/pkg/engine/provider/extract.go

  Line 132 in fdea929

  getterDst, err := getPaths(&goGetter)

  ) could fail for several reasons, among other thing a bug of go-getter. what about skipping any path that we fail to resolve?

[lior-orca]

@pereiramarco011 any updates? what is preventing us from pushing the change?

[pereiramarco011]

@pereiramarco011 any updates? what is preventing us from pushing the change?

Hello Lior, I hope all is well.

The development is complete, all that is left to do is the review and QA, it is programmed to be delivered with KICS version 1.7.7, should be out in less than a week if all goes well.

Thank you for all the help,

Best Regards,
Marco from KICS Team