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

Config file needs documentation 🧐 #136

Closed
tgotwig opened this issue Aug 8, 2020 · 4 comments
Closed

Config file needs documentation 🧐 #136

tgotwig opened this issue Aug 8, 2020 · 4 comments

Comments

@tgotwig
Copy link
Contributor

tgotwig commented Aug 8, 2020

When running rustscan, one of the first messages is:

[~] The config file is expected to be at "/Users/tgotwig/Library/Application Support/rustscan/config.toml"

But when I search for config inside the readme I get nothing? 🤔

Describe the solution you'd like
Config file should be explained inside the readme. 🧐

@issue-label-bot
Copy link

Issue Label Bot is not confident enough to auto-label this issue. See dashboard for more details.

@bee-san
Copy link
Member

bee-san commented Aug 8, 2020

Hey @tgotwig ! You've stumbled upon a mysterious new feature 😉

I was going to implement this, but then I got side tracked by #131 haha.

The config file isn't implemented yet (see my unfinished PR at #126)

Since you tried to search for the config file, what was you expecting it to do? What would you like it to do? We're still working on this and community support is v/ welcomed 😸

See issue #71 for more about config. In short at the moment it will be:

  • A list of IP addresses to scan
  • An adaptive learning based top X ports. So it learns what ports you most frequently scans, and you can request top 1k ports, top 250 ports and so on.

If you have any suggestions, please leave a comment. Since I finished the pretty interface I am working on this now :D

@tgotwig
Copy link
Contributor Author

tgotwig commented Aug 8, 2020

Oh nice so it's already work in progress 🙂

Not sure what to expect, just used it for a few minutes and thought this could be worth an issue 😁 maybe for overwriting default command-line-arguments like batch-size 🤓

@bee-san
Copy link
Member

bee-san commented Aug 8, 2020

Cool idea! I'll add this 🧑‍💻

@bee-san bee-san closed this as completed Aug 10, 2020
@andy5995 andy5995 mentioned this issue Aug 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants