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

Fixed multiple issues #630

Merged
merged 4 commits into from May 14, 2024
Merged

Fixed multiple issues #630

merged 4 commits into from May 14, 2024

Conversation

MrTiz
Copy link
Contributor

@MrTiz MrTiz commented Oct 15, 2022

  1. Added category_uuid to _filterset_tags in tenable/io/tags.py
  2. Added support for encrypted NessusDB reports
  3. Fixed bug when uploading an encrypted file

@MrTiz MrTiz requested a review from a team October 15, 2022 08:48
@MrTiz MrTiz changed the title Added 'category_uuid' to '_filterset_tags' in 'tenable/io/tags.py' Fixed multiple issues Mar 15, 2023
@aseemsavio
Copy link
Collaborator

There seems to be failing tests..

@MrTiz
Copy link
Contributor Author

MrTiz commented Mar 17, 2023

There seems to be failing tests..

Hi @aseemsavio, the code I modified in the file file.py works, I tested it myself, the problem is in the unit test, however I am not able to solve it. The current version of the file file.py does not work correctly when trying to load an encrypted file. You can test it yourself by trying to use the pytenable library to do the import of an encrypted Nessus DB report, you will get error 500 Internal Server Error returned by Tenable.io.

Please help me to fix this bug. Thanks.

@SteveMcGrath
Copy link
Contributor

@MrTiz may I suggest the following change to the test suite to allow it to pass?

a3b3be1

MrTiz added 4 commits May 14, 2024 09:44
Added 'category_uuid' to _filterset_tags
Added support for encrypted NessusDB reports
According to the official documentation (https://developer.tenable.com/reference/file-upload), the 'no_enc' parameter must be in the url, not in the body.
@SteveMcGrath SteveMcGrath merged commit 30bf0f1 into tenable:master May 14, 2024
8 of 15 checks passed
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.

None yet

3 participants