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

Project updates for up-to-date usage #41

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

FoxxMD
Copy link

@FoxxMD FoxxMD commented May 2, 2024

Changes I had to make in order to use netbox-scanner today

FoxxMD added 3 commits May 2, 2024 09:24
Clarifies format of address as protocol is required for python requests module to work
The existing requirements were not satisfiable. Removing versions causes pip to resolve automatically which still works with the project
@mariano-daniel
Copy link

Hello @FoxxMD ! Is it still working for you? I'm trying by all means to get it working. I can generate the XML files but no luck uploading them to Netbox.

@FoxxMD
Copy link
Author

FoxxMD commented Jul 29, 2024

Sorry but I'm not using netbox anymore so I can't test this.

I vaguely remember I had problems with the upload/processing step because the program uses a relative path for searching for the xml files -- so where you run ./netbox-scanner.py determines where it looks for the files, I think. It might be that the xml folder needs to be in the same location as netbox-scanner.py or that where you run the program determines the location path, I'm not sure.

@mariano-daniel
Copy link

Thanks @FoxxMD ! Is the XML folder the Scans folder?

This is what my directory looks like:

/opt/netbox-scanner/netbox-scanner-master# ls
LICENSE    logs  netbox-scanner.py  nmap-scan.sh      samples  tests
README.md  nbs	 networks.txt.bak   requirements.txt  scans

@FoxxMD
Copy link
Author

FoxxMD commented Jul 30, 2024

yes i think the scans folder is correct

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

Successfully merging this pull request may close these issues.

2 participants