Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Own configuration file #71

Closed
cavo789 opened this issue Oct 3, 2022 · 0 comments · Fixed by #75
Closed

Own configuration file #71

cavo789 opened this issue Oct 3, 2022 · 0 comments · Fixed by #75
Labels
enhancement New feature or request hacktoberfest

Comments

@cavo789
Copy link

cavo789 commented Oct 3, 2022

Hello

Is it possible to use my own configuration file for, only, exceptions i.e. I want to keep yours but f.i. I wish to be able to skip anything under /var/www/vendor (excluded folder).

As far I understand, I can create a config.yaml file and use it with --config-path but if I do this, your file won't be loaded anymore so I first need to copy/paste yours in mine and make some changes (https://github.com/deepfence/SecretScanner/blob/master/config.yaml)

This is bad since yours will be upgraded in the future for more rules so, on my side, I just need to be able to extend it with my own rules like my exclusions.

Also, is it possible to foresee/change the --config argument so I can provide a filename (like .config/secret-scanner.yml. The name 'config.yaml is too generic; I've already plenty of configuration files.

Thanks

@ibreakthecloud ibreakthecloud added enhancement New feature or request good first issue Good for newcomers hacktoberfest and removed good first issue Good for newcomers labels Oct 6, 2022
ramanan-ravi pushed a commit that referenced this issue Oct 17, 2022
…on. (#75)

* Allow for merging multiple config files.

* Handle files specified as config paths.

Closes #71
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request hacktoberfest
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants