Bind nginx load balancer to all interfaces (0.0.0.0) (#11)

[t0mdavid-m]

Change the nginx listen directive from listen 8501 to listen 0.0.0.0:8501 so the load balancer explicitly accepts connections on all network interfaces, not just the default.

https://claude.ai/code/session_01721rHYJhSZfJnnUpWHXbVs

Summary by CodeRabbit

• Chores
  • Updated service network configuration to enable external access to the application, allowing it to be reached from outside the local machine.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The Dockerfile's nginx configuration was modified to bind the upstream server and server listen directives to all network interfaces (0.0.0.0:8501) instead of localhost only (8501). This change exposes the Streamlit instances to external traffic when running multiple servers.

Changes

Cohort / File(s)	Summary
Nginx Configuration Dockerfile	Modified upstream and server listen directives from 8501 to 0.0.0.0:8501, enabling external network access to Streamlit instances.

Poem

🐰 The firewall opens wide today,
No longer locked in localhost's way,
All interfaces now hear the call,
At port eight-five-oh-one for all! 🌐

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix_ip

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}