search

SecureSECO / searchSECO-miner

v2 miner implementation for SearchSECO
https://docs.secureseco.org/searchseco-integration
MIT License
2 stars 3 forks source link

Add srcML Warning #47

Closed slingerbv closed 1 week ago

slingerbv commented 2 months ago

If srcML is not installed, we should get a warning. Otherwise, a lot of times only 0 methods will be found and the tool is as good as useless.

slingerbv commented 2 months ago

ChatGPT suggests fix here: https://chatgpt.com/share/ae6cee74-78ce-4ed2-b1a9-85789615585c

But please do not stop the mining process, just warn loudly.

slingerbv commented 2 months ago

Change the line to: Logger.Warning(srcML is not available. We will only mine code for which we have our own parsers, which are PHP and Python., Logger.GetCallerLocation());

And then don't return, because then it will not create a miner.

slingerbv commented 1 week ago

This was apparently already fixed!