Skip to content

[Butler] Make local App Engine instance accessible from local network - #5422

Open
JuanMBriones wants to merge 1 commit into
masterfrom
local_making_instance_accessible_from_local_network
Open

[Butler] Make local App Engine instance accessible from local network#5422
JuanMBriones wants to merge 1 commit into
masterfrom
local_making_instance_accessible_from_local_network

Conversation

@JuanMBriones

Copy link
Copy Markdown

When developing locally or interacting with the ClusterFuzz web server from other devices on the local network (such as another workstation or test devices), the development App Engine server was bound only to 127.0.0.1, preventing external access on the LAN.

  • Added DEV_APPSERVER_ADDR = '0.0.0.0' in constants.py and updated DEV_APPSERVER_HOST to use this address.

  • Updated run_server.py to bind gunicorn to DEV_APPSERVER_HOST instead of hardcoded 127.0.0.1.

  • Ran python butler.py lint (passed).

  • Ran python butler.py integration_tests (passed).

  • Ran python butler.py py_unittest -t core -m (passed).

  • Ran python butler.py py_unittest -t appengine -m (passed).

When developing locally or interacting with the ClusterFuzz web server from
other devices on the local network (such as another workstation or test
devices), the development App Engine server was bound only to `127.0.0.1`,
preventing external access on the LAN.

- Added `DEV_APPSERVER_ADDR = '0.0.0.0'` in `constants.py` and updated
  `DEV_APPSERVER_HOST` to use this address.
- Updated `run_server.py` to bind gunicorn to `DEV_APPSERVER_HOST` instead
  of hardcoded `127.0.0.1`.

- Ran `python butler.py lint` (passed).
- Ran `python butler.py integration_tests` (passed).
- Ran `python butler.py py_unittest -t core -m` (passed).
- Ran `python butler.py py_unittest -t appengine -m` (passed).

Signed-off-by: Manuel Briones <manuelbriones@google.com>

@dylanjew dylanjew left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did this work with the developer containers?

## Development Containers (Recommended)

@JuanMBriones

Copy link
Copy Markdown
Author

did this work with the developer containers?

## Development Containers (Recommended)

Hiii Dylan, thanks for the review!!!:D

When working with devcontainers VSCode does the binding automatically (it makes available the running instance on the local network as well). That's why this is only focused when running the service through butler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants