Skip to content

Harden lyrics search and resolve audit findings - #149

Merged
variablenix merged 1 commit into
mainfrom
audit/lyrics-production-hardening
Aug 30, 2026
Merged

Harden lyrics search and resolve audit findings#149
variablenix merged 1 commit into
mainfrom
audit/lyrics-production-hardening

Conversation

@variablenix

Copy link
Copy Markdown
Owner

Summary

  • harden Genius authentication, redirect handling, result URL validation, and IRC-safe output formatting
  • preserve one complete Genius lyrics link, including a bounded short-link fallback for unusually long canonical URLs
  • add regression coverage for all reported Electric Wizard and Snoop Dogg query forms and upstream failure modes
  • resolve repository-wide static-analysis findings discovered during the audit
  • document the lyrics output and token-handling guarantees

Validation

  • go test -count=1 -race ./...
  • go vet ./...
  • go build ./...
  • go test -cover ./...
  • staticcheck ./...
  • govulncheck ./... (no vulnerabilities found)
  • focused gosec scan (0 actionable issues after reviewing documented false-positive categories)
  • go mod verify
  • go mod tidy -diff
  • git diff --check

@variablenix
variablenix merged commit f3c922a into main Aug 30, 2026
3 checks passed
@variablenix
variablenix deleted the audit/lyrics-production-hardening branch August 30, 2026 06:19
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