Repository navigation
fix: correct HTTP target parsing - #205
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (5)
Included review availability: This review used your included allowance. 3 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. WalkthroughHTTP target parsing now supports HTTP/2 URLs, bare IP addresses, and escaped paths. IPv6 URL hosts must be bracketed. Measurement request construction and IP-target detection use the updated parser. The exported Suggested reviewers: Priority: ➖ Normal Merge Risk: ⚪ Minimal · up to The changed request path preserves encoded slashes as intended. No actionable merge-blocking risk remains after normal checks. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Comment |
|
🎉 This PR is included in version 1.6.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Addresses jsdelivr/globalping.io#203 (comment)
Percent-encoded URL paths are no longer decoded before creating HTTP measurements. For example, /a%2Fb now requests /a%2Fb instead of /a/b; use /a/b explicitly to retain the previous request path. IPv6 URL hosts must be bracketed. Bare IPv6 addresses can now be used directly as targets