Skip to content

Corrected Regression created from scrub utility #1006#1007

Merged
SteveMcGrath merged 2 commits into
mainfrom
fix/scans-endpoint-regression
Jun 9, 2026
Merged

Corrected Regression created from scrub utility #1006#1007
SteveMcGrath merged 2 commits into
mainfrom
fix/scans-endpoint-regression

Conversation

@SteveMcGrath

Copy link
Copy Markdown
Contributor

Description

scrub utility introduced as part of the restfly v1 vendoring process introduced a regression in the scans endpoint as it was overly restrictive on the allowed characters for the path.

Fixes #1006

Type of change

Please delete options that are not relevant.

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

How Has This Been Tested?

  • Added new tests to confirm That these newer formats are unaffected.
  • Added new tests to confirm warning logs are emitted when scrubbing has occured.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@SteveMcGrath SteveMcGrath requested a review from a team as a code owner June 9, 2026 18:29

@amaruf-tenb amaruf-tenb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@SteveMcGrath SteveMcGrath force-pushed the fix/scans-endpoint-regression branch from 43893da to 97e24be Compare June 9, 2026 22:04
@SteveMcGrath SteveMcGrath merged commit 9074625 into main Jun 9, 2026
10 checks passed
@SteveMcGrath SteveMcGrath deleted the fix/scans-endpoint-regression branch June 9, 2026 22:07
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.

scans.export() fails with 404 when Tenable cloud returns new SSE-* / SSEF-* file IDs after upgrading from 1.9.1 to 26.5.1

2 participants