Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: Document HTTP proxy configuration for Edge Proxy #4928

Merged
merged 3 commits into from
Dec 20, 2024

Conversation

rolodato
Copy link
Member

@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
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
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
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.

2 participants