pathetiq / BurpSmartBuster

A Burp Suite content discovery plugin that add the smart into the Buster!
MIT License
383 stars 68 forks source link

Implement IScannerCheck #15

Closed ahri closed 7 years ago

ahri commented 7 years ago

IScannerCheck was not fully implemented. This can be patched in with dummy implementations.

This should result in a BApp that doesn't need any manual installation of dependencies, because they're now declared in a requirements.txt file in the root of the repo (in our fork). However, this is not quite the case:

**********************************************************************
  Resource u'tokenizers/punkt/english.pickle' not found.  Please
  use the NLTK Downloader to obtain the resource:  >>>
  nltk.download()
  Searched in:
    - '/home/adam/nltk_data'
    - '/usr/share/nltk_data'
    - '/usr/local/share/nltk_data'
    - '/usr/lib/nltk_data'
    - '/usr/local/lib/nltk_data'
    - u''
**********************************************************************

BApps should not need manual download steps, in order to ease their use by potentially un-knowledgeable users.

Please contact me on Twitter - @bitwog or via email - adam.piper@portswigger.net to discuss this.