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.

Sign up for GitHub

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

Jump to bottom

feat(proxy-gae): Generalise proxy-gae to forward generic routes #149

Merged
merged 32 commits into from
Nov 25, 2022
Merged

feat(proxy-gae): Generalise proxy-gae to forward generic routes #149

merged 32 commits into from
Nov 25, 2022

Conversation

mrzzy
Copy link
Owner

@mrzzy mrzzy commented Nov 25, 2022 •
edited
Loading

Motivation

Generalise proxy gae so its able act as a proxy as needed instead of just proxying WARP VM.

Contents

Generalise proxy-gae to forward generic routes:

  • embed a template.py that runs on proxy-gae's container entrypoint. This script templates a nginx.conf based the content of PROXY_SPEC env var. (eg. /proxy=http://proxy.me creates a location block to forward /proxy/here to http://proxy.me/here)
  • generalise terraform deployment for proxy-gae from being WARP VM specific.

@mrzzy
refactor(terraform): extract out gae_proxy gae service into its own t…
718137f 
…f module
@mrzzy
build(proxy): track jinja2 dependency in requirements.txt
5a1f266
@mrzzy
chore(pre-commit): add black pre-commit hook to format python code
1f817b9
@mrzzy
feat(proxy-gae): add template.py to template nginx.conf from proxy spec
e0ccf3c
@mrzzy
refactor(proxy-gae): remove old nginx templates solution
3c05b65
@mrzzy
chore(git): ignore Session.vim
5266ce7
@mrzzy
refactorg(gae_proxy): rename gae_proxy tf module to proxy_gae to alig…
670c93d 
…n naming
@mrzzy
refactor(proxy_gae): update container to template nginx.conf from PRO…
8ca9814 
…XY_SPEC
@mrzzy
refactor(terraform): proxy_gae tf module to support 'proxy_spec'
4c92030
@mrzzy
fixup! feat(proxy-gae): add template.py to template nginx.conf from p…
f2c8d66 
…roxy spec
@mrzzy
refactor(terraform): rename resources to reflect generic proxy
0ff03af
@mrzzy
ci(docker): lint python with black
d160c76
@mrzzy
refactor(proxy-gae): remove old nginx.conf
f614f61
@mrzzy
refactor(terraform): generalize proxy-gae from being warp vm specific
3b975e8
@mrzzy
feat(proxy-gae): standardize handling of trailing slashes in proxy spec
a07c9e7
@mrzzy
fix(proxy-gae): leading slash in proxy spec being stripped
34c8a7b
@mrzzy
fix(proxy-gae): /route prefix included in proxied url
18fec71
@mrzzy
refactor(docker): copy only required files into proxy-gae container
ef46c78
@mrzzy
build(proxy-gae): add testcontainers to dev dependencies
25bab5c
@mrzzy
test(proxy-gae): add end to end test
7c1310b
@mrzzy
fix(proxy-gae): proxy-gae unable to container names in e2e test
af53d2b 
proxy-gae was hardcoded to resolve with cloudflare dns, which will be
unable to resolve the containers from docker-compose
@mrzzy
ci(docker): run proxy-gae e2e test before publishing container
734b19c
@mrzzy mrzzy added the feature New feature or request label Nov 25, 2022
@mrzzy mrzzy self-assigned this Nov 25, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 25, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mrzzy mrzzy merged commit cbadf0c into main Nov 25, 2022
@mrzzy mrzzy deleted the feat/generic-proxy branch November 25, 2022 03:10
@mrzzy mrzzy mentioned this pull request Feb 25, 2023
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mrzzy mrzzy

Labels
feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@mrzzy