Skip to content

A scan target URL is printed, reported and saved without its user information - #62

Merged
thebenignhacker merged 1 commit into
mainfrom
integrate/2026-10-22-a
Oct 22, 2026
Merged

thebenignhacker merged 1 commit into
mainfrom
integrate/2026-10-22-a

Conversation

@thebenignhacker

Copy link
Copy Markdown
Member

This pull request lands 1 commit in internal/cli and CHANGELOG.md.

Included

  • A scan target URL is printed, reported and saved without its user information

Verified on this exact tree

  • go build -v ./...: exit 0
  • go vet ./...: exit 0
  • go test -race -count=1 -p 4 -json ./...: exit 0; 9 packages, 8 passed and 1 with no test files (pkg/version); 62 test files (git ls-files '*_test.go' | wc -l); 377 top-level tests passed, 0 failed, 0 skipped; 842 subtests passed, 0 failed, 0 skipped
  • Changelog check, the 4 tests in pkg/reporter/changelog_test.go within the run above: 4 passed, 0 failed
  • golangci-lint run --max-same-issues 0 --max-issues-per-linter 0 ./... (golangci-lint 2.12.2): exit 1, 89 issues (errcheck 49, staticcheck 40); pre-existing: the same command on origin/main (b2b4d63) reports the same 89 issues, compared by file, linter and message

Not included

None

Closes

None

How to merge

Rebase and merge, so that each change stays its own commit on main.

…ormation

The JSON report's scanTarget, the scanTarget read from a --baseline file,
the text and pqc reports, the line printed when no file was scanned, and an
error that quotes the target name a repository URL without the user name
and password in it. The clone uses the URL as it was typed.
@thebenignhacker
thebenignhacker merged commit ff7ca21 into main Oct 22, 2026
4 checks passed
@thebenignhacker
thebenignhacker deleted the integrate/2026-10-22-a branch October 22, 2026 04:09
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.

1 participant