* enhancement: supporting http tunneled rtsp
* refactor: simplify HTTP tunnel support per review feedback
- Extract streamCandidate() for nmap port classification
- Add isCommonHTTPPort() for masscan and nmap fallback
- Move URL building to Stream.String() and Stream.URL()
- Pass Stream directly to attack methods instead of individual args
- Add TLS config for HTTPS tunnel support
- Make auth detection non-fatal for tunneled streams
- Rename HTTPTunnel to UseHTTPTunnel
* - Testing the auth workflow for the tunneled streams is not blocking the rest of the pipeline since I changed the return values to Auth unknown and nil
- added extra ports in the test according to suggestions
* fixing some lint errors
* removing the unused buildrtspurl
* delayed the urlstream call for clarity
removed error messages
refactored the test that used the deprecated buildTRSPurl to use stream.URL and stream.String() methods
* extracting iscommonHTTP port to pkg/ports (package ports)
switching on u.scheme to create proper schemes for http and https
* refactor: replace HTTP tunnel bool with scheme-based detection; enable TLS only for HTTPS-tunneled streams
* chore: simnplify InferTunnelScheme and newRTSPClient
* fix: remove rendundant check in streamCandidate
* fix: typo in parseScheme
* tests: coverage for new schemes
* fix: use RTSP and not RTSPS for HTTPS URLs
* fix: tunneled RTSP scheme handling and auth detection fallback
* ui: render empty credentials as none in summary and TUI
* chore: ignore duplicate-string warning for none literal
* Potential fix for pull request finding
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
* fix: rtsps probe headers
---------
Co-authored-by: Brendan Le Glaunec <brendan@glaulabs.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Lawrence Arryl Lopez
8531c006d4