update vite.config to allow all hosts

[BuckyMcYolo]

Summary

This PR adds a preview configuration block to apps/web/vite.config.ts with allowedHosts: true. This setting allows the Vite preview server to accept requests from any host, removing the default localhost-only restriction.

Changes

• File: apps/web/vite.config.ts
• Lines changed: +3
• Modification: Added preview: { allowedHosts: true } configuration

Analysis

Context:
This change is part of a broader effort to support Docker/containerized deployments. A prior commit (20bec18) attempted a similar fix using the --host command-line flag; this PR takes a configuration-based approach instead.

Strengths:

• Minimal, focused change that doesn't affect build logic
• Uses a legitimate Vite configuration option
• Syntactically correct and won't cause runtime errors
• Appears to address a real deployment issue with containerized environments

Concerns:

• Security implications: allowedHosts: true permits requests from any host without restrictions. This is permissive and could expose the preview server in production if not properly gated by other means
• Lack of context: No PR description explaining the use case or deployment scenario
• Environment-agnostic: No conditional logic based on environment (e.g., development vs. production). If this config is used in production, it represents a potential security exposure
• Limited scope: Only addresses the preview server; it's unclear if there are other host configuration needs for dev or build servers
• No documentation: No updates to README or documentation explaining the change or its implications

Confidence Score: 3/5

The PR is technically sound and won't break the codebase, but the broad host allowance raises moderate security concerns without sufficient context about the intended deployment environment. The change would benefit from: (1) a clear explanation of the use case and deployment scenario, (2) conditional configuration based on environment, and (3) documentation clarifying security implications. Acceptable for merge with a follow-up explaining the deployment context.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: f3d01e5b-aff5-456c-9859-b8f1bf3e8026

📥 Commits

Reviewing files that changed from the base of the PR and between cb6152f and e90a05e.

📒 Files selected for processing (1)

• apps/web/vite.config.ts

---

📝 Walkthrough

Walkthrough

The Vite configuration was updated to allow the preview server to accept requests from any host by adding a preview configuration block with allowedHosts: true. This is a configuration-only change with no logic or entity signature modifications.

Changes

Cohort / File(s)	Summary
Vite Preview Configuration apps/web/vite.config.ts	Added preview configuration block with allowedHosts: true to permit preview server requests from any host.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• feat: added dockerfile to web app and fixed vite config to use --host #34: Modifies Vite preview server host acceptance by adding --host flag to the preview start script; complements this PR's configuration-based approach.

Poem

🐰 With hosts now free to roam and play,
The preview server brightens the day,
Three little lines, so small yet grand,
Any connection, from any land! 🌐

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch dev

---

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