Refinement: Removed Redundancy of Scan Config Passing

[aryanagrawal22]

Description

• Issue: https://github.com/google/gcp_scanner/issues/241. Hence I created a property crawler_config in base class ICrawler which is returns bool value, where default=False
• If there is a need or optionality of passing the config to the crawler then this property crawler_config in the base class needs to be overridden in the sub class to return True.

Note : Currently as I was checking all the crawlers, I have just seen that config is being used only inside the StorageBucketsCrawler crawler. So only its value is set as True in its base class.

Changes Made

• Base class ICrawler has a new property called as crawler_config which returns the abstracted private variable _config_depndency which is set to False by default.
• StorageBucketsCrawler class has overridden crawler_config to return True.
• get_crawl function in scanner.py has a check now calling the crawler_config to decide to send config or not.

Testing

Local testing done with logs denoting that where config file is sent and where it is not. (Removed the logs after the testing)

This check denotes that only StorageBucketsCrawler is being sent the config file

[aryanagrawal22]

Thanks for your PR. I left some suggestions. PTAL.

Thank you for going through my PR, and reviewing it. I think, I have done the relevant changes as asked. Please let me know in case any others are needed.

[aryanagrawal22]

Yes, fair point. I have made the changes, changed the name to has_config_dependency from crawler_config

Pushed with the changes 😁