Skip to content

Add Maven artifact resource support for SpotBugs filter files#1384

Draft
Copilot wants to merge 5 commits intomasterfrom
copilot/add-support-for-artifactory-files
Draft

Add Maven artifact resource support for SpotBugs filter files#1384
Copilot wants to merge 5 commits intomasterfrom
copilot/add-support-for-artifactory-files

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 18, 2026

  • Understand issue Feature Request: Support resolving excludeBugFile/excludeFilterFile/includeFilterFile from remote URLs with authentication #1375 (HTTPS URL + filterServerId)
  • Review what the previous task implemented (mvn: scheme — interesting but not what was asked)
  • Extend ResourceHelper to fetch http/https URLs with optional Basic auth
  • Add filterServerId parameter to SpotBugsMojo; inject Settings + SettingsDecrypter to look up credentials
  • Fix sanitization test (was using http URL which now triggers real HTTP fetch)
  • Add tests for HTTP URL fetching (with and without auth, and failure cases)
  • Update docs (usage.apt.vm)
  • Run full test suite — 176 pass, 2 skipped, 0 failures
  • Parallel validation — pass; addressed: regex docs, null-decrypter log warning

@sonarqubecloud
Copy link
Copy Markdown

Copilot AI requested a review from hazendaz April 18, 2026 19:02
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.

2 participants