Skip to content
This repository was archived by the owner on May 14, 2021. It is now read-only.

Bug: DemonstrateVulnerabilityAPI() #31

Merged
gschneider-r7 merged 2 commits into
masterfrom
bug/encoding_fix
Oct 4, 2017
Merged

Bug: DemonstrateVulnerabilityAPI() #31
gschneider-r7 merged 2 commits into
masterfrom
bug/encoding_fix

Conversation

@ghost

@ghost ghost commented Oct 4, 2017

Copy link
Copy Markdown

Description

Workaround for the following error

nexpose.nexpose.NexposeConnectionException: Unable to execute the request: Unicode strings with encoding declaration are not supported. Please use bytes input or XML fragments without declaration.!

Observed when editing run_demo.py and uncommenting DemonstrateVulnerabilityAPI()

Motivation and Context

DemonstrateVulnerabilityAPI() demo does not work as is

How Has This Been Tested?

Ran run_demo.py with DemonstrateVulnerabilityAPI() uncommented.

Note Currently unable to test against all of the Demonstration methods as some appear to be broken;

Working

  • DemonstrateTagAPI()
  • DemonstrateAssetAPI()
  • DemonstrateVulnerabilityAPI()
  • DemonstrateSiteAPI()
  • DemonstrateAssetGroupAPI()
  • DemonstrateVulnerabilityExceptionAPI()
  • DemonstrateRoleAPI()
  • DemonstrateEngineAPI()
  • DemonstrateScanPI()
  • DemonstrateTicketAPI()

Not-working

  • DemonstrateBackupAPI()
  • DemonstrateCriteriaAPI()
  • DemonstrateSharedCredentialsAPI()
  • DemonstrateAssetFilterAPI()
  • DemonstrateDiscoveryConnectionAPI()
  • DemonstrateUserAPI()

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have updated the documentation accordingly (if changes are required).
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@ghost ghost requested a review from gschneider-r7 October 4, 2017 16:22
@ghost ghost added the bug label Oct 4, 2017
@gschneider-r7 gschneider-r7 merged commit 0282a1b into master Oct 4, 2017
@gschneider-r7 gschneider-r7 deleted the bug/encoding_fix branch October 5, 2017 00:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant