Skip to content

Conversation

@rolodato
Copy link
Contributor

@rolodato rolodato commented Dec 18, 2024

See Flagsmith/edge-proxy#134

Changes can be seen at https://docs-git-docs-edge-proxy-http-proxy-flagsmith.vercel.app/deployment/hosting/locally-edge-proxy#environment-variables

Confirmed with:

% cat tinyproxy.conf
Port 8889
Timeout 600
Listen 0.0.0.0
% tinyproxy -d -c tinyproxy.conf
% docker run \
    -e ALL_PROXY="host.docker.internal:8889" \
    -p 8000:8000 \
    flagsmith/edge-proxy:latest

Result:

CONNECT   Dec 18 13:22:25.809 [22667]: Connect (file descriptor 4): 127.0.0.1
CONNECT   Dec 18 13:22:25.809 [22667]: Request (file descriptor 4): CONNECT edge.api.flagsmith.com:443 HTTP/1.1
INFO      Dec 18 13:22:25.809 [22667]: No upstream proxy for edge.api.flagsmith.com
INFO      Dec 18 13:22:25.809 [22667]: opensock: opening connection to edge.api.flagsmith.com:443
INFO      Dec 18 13:22:25.817 [22667]: opensock: getaddrinfo returned for edge.api.flagsmith.com:443
CONNECT   Dec 18 13:22:25.823 [22667]: Established connection to host "edge.api.flagsmith.com" using file descriptor 8.
INFO      Dec 18 13:22:25.823 [22667]: Not sending client headers to remote machine
INFO      Dec 18 13:22:27.485 [22667]: Closed connection between local client (fd:4) and remote client (fd:8)

@rolodato rolodato requested a review from a team as a code owner December 18, 2024 16:23
@rolodato rolodato requested review from zachaysan and removed request for a team December 18, 2024 16:23
@vercel
Copy link

vercel bot commented Dec 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
flagsmith-frontend-preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 2:15pm
flagsmith-frontend-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 2:15pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview Dec 20, 2024 2:15pm

@github-actions github-actions bot added docs Documentation updates and removed docs Documentation updates labels Dec 19, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2024

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-4928 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-4928 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api:pr-4928 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-frontend:pr-4928 Finished ✅ Results
ghcr.io/flagsmith/flagsmith:pr-4928 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-private-cloud:pr-4928 Finished ✅ Results

@github-actions github-actions bot added docs Documentation updates and removed docs Documentation updates labels Dec 19, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2024

Uffizzi Ephemeral Environment deployment-59251

☁️ https://app.uffizzi.com/github.com/Flagsmith/flagsmith/pull/4928

📄 View Application Logs etc.

What is Uffizzi? Learn more!

@github-actions github-actions bot added docs Documentation updates and removed docs Documentation updates labels Dec 19, 2024
@matthewelwell matthewelwell force-pushed the docs/edge-proxy-http-proxy branch from 68b00b9 to d6261a5 Compare December 19, 2024 15:30
@matthewelwell
Copy link
Contributor

@rolodato now that #4932 is merged, can you rebase your PR onto main? That should get CI passing.

@matthewelwell matthewelwell merged commit 1a1235e into main Dec 20, 2024
11 checks passed
@matthewelwell matthewelwell deleted the docs/edge-proxy-http-proxy branch December 20, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants