Skip to content

chore: enforce HTTPS-only URL validation and stage logging - #1069

Open
kireetivar wants to merge 2 commits into
fortify:feat/v3.x/aviator/26.4from
kireetivar:p/kireetivar/bug_fixes
Open

chore: enforce HTTPS-only URL validation and stage logging#1069
kireetivar wants to merge 2 commits into
fortify:feat/v3.x/aviator/26.4from
kireetivar:p/kireetivar/bug_fixes

Conversation

@kireetivar

Copy link
Copy Markdown
Contributor

This PR Improves Aviator connection diagnostics by enforcing HTTPS endpoints and adding clearer stage-level logging.

Changes

  • Enforce HTTPS-only Aviator URLs.
  • Continue supporting scheme-less hosts by normalizing them to https://.
  • Normalize uppercase HTTPS schemes to lowercase.
  • Reject unsupported schemes such as http, ftp, and invalid scheme values with a dedicated exception.
  • Add shared URL scheme detection for Aviator and proxy handling.
  • Include URL scheme and provided URL details in endpoint failure evidence.
  • Add diagnostic start and completion logging for endpoint, DNS, TCP, proxy, TLS, gRPC, and credential stages.
  • Improve PASS, FAIL, WARN, and skipped-stage log messages.
  • Include resolved DNS addresses and exception details in diagnostic logs where available.
  • Preserve structured skip reasons in diagnostic results.
  • Add and update regression tests for URL validation, diagnostic reporting, stage logging, and proxy handling.

Testing

  • Added and updated focused unit tests for Aviator URL normalization and connection diagnostics.

@kireetivar kireetivar self-assigned this Aug 5, 2026
@kireetivar
kireetivar marked this pull request as ready for review August 5, 2026 08:16
@kireetivar
kireetivar requested a review from rsenden August 5, 2026 08:29
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