Allow list global scan

[AnthonyPoschen]

Description:

the changes are a proposal to address #911 they have been tested with real workloads and do resolve the example provided in the issue #911 I am not sure if these variable names are a good fit, Happy to take suggestions and modify the PR accordingly.

Checklist:

• [ Y ] Does your PR pass tests?
• [ N ] Have you written new tests for your changes?
• [ Y ] Have you lint your code locally prior to submission?

[jit-ci[bot]]

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

All security workflows are defined in a centralized repository named .jit. In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

[AnthonyPoschen]

@zricethezav i modified the code to use the output from location helper function to get the string segment of the line to avoid multi line matches.

Maybe my idea of considering multi line matching should be left to a valid use case scenario in another ticket to address.

[AnthonyPoschen]

had an idea where instead of a boolean we can have a int to determine level 0 == secret 1 == full line 2 == multi line

Thoughts @zricethezav

[jqgeek]

I also encountered this kind of problem. Will the function of this problem be updated later? @zricethezav

[AnthonyPoschen]

@zricethezav any guidance on next steps? i made the recommended modifications from the review

[AnthonyPoschen]

made changes to clarify this is against the full match rather than single line