fix: explicitly set ProxyConnectHeader for Webshare HTTPS CONNECT auth#9
Closed
Jing-yilin wants to merge 3 commits into
Closed
fix: explicitly set ProxyConnectHeader for Webshare HTTPS CONNECT auth#9Jing-yilin wants to merge 3 commits into
Jing-yilin wants to merge 3 commits into
Conversation
…crawl
Backend:
- CampaignSnapshot table for 24h velocity calculation
- Campaign.velocity_24h + pledge_delta_24h fields
- Alert.alert_type ('keyword'|'momentum') + velocity_thresh field
- Cron: RunCrawlNow (exported), snapshot storage, velocity computation
- ListCampaigns: sort=hot from DB, DB fallback when GraphQL fails
- Startup goroutine triggers RunCrawlNow 3s after server boot
- Improved Cloudflare bypass headers on session bootstrap
iOS:
- CampaignDTO: velocity_24h, pledge_delta_24h, first_seen_at fields
- CampaignRowView: momentum badge (⚡ +N% / 🔥 +N%) and New badge
- DiscoverView: 🔥 Hot sort option
- AlertsView: Momentum alert type with velocity threshold slider
…ter requests - Add WEBSHARE_PROXY_URL config read from env/Secrets Manager - Pass proxy to KickstarterGraphClient and KickstarterRESTClient http.Transport - Disable ForceAttemptHTTP2 (incompatible with HTTP CONNECT proxy) - Add webshare-proxy-url to Secrets Manager ARN resolution in deploy workflow - Update .env.example with WEBSHARE_PROXY_URL documentation
Go's http.Transport does not auto-retry 407 for CONNECT tunnels. Must set Proxy-Authorization header upfront via ProxyConnectHeader.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Go's
http.Transportdoes not automatically handle 407 challenge-response for HTTPS CONNECT tunneling. This fix setsProxyConnectHeaderwith theProxy-Authorizationheader upfront so Webshare residential proxy auth works from ECS.Fixes the
Proxy Authentication Requirederror introduced in #8.