docs(enterprise): add path-based sandbox routing documentation#486
docs(enterprise): add path-based sandbox routing documentation#486jpshackelford wants to merge 4 commits intomainfrom
Conversation
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟢 Good taste - Clear, well-structured documentation that addresses a real enterprise deployment constraint.
[RISK ASSESSMENT]
- [Overall PR]
⚠️ Risk Assessment: 🟢 LOW
Documentation-only change addressing verified enterprise requirement (wildcard certificate restrictions). Uses Mintlify components correctly, provides clear guidance for both routing modes, and maintains consistent terminology throughout. No code modifications.
VERDICT:
✅ Worth merging: Solid documentation that solves a real problem.
KEY INSIGHT:
Tabbed structure elegantly separates two distinct deployment modes while keeping the documentation DRY and maintainable.
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
jlav
left a comment
jlav
left a comment
There was a problem hiding this comment.
Left some nits, but overall LGTM.
jlav
left a comment
jlav
left a comment
There was a problem hiding this comment.
Left some nits, but overall LGTM.
Update the enterprise quick-start guide to document the path-based sandbox routing feature introduced in OpenHands-Cloud PR #563. Key changes: - Add tabbed DNS/TLS setup section covering both routing modes - Document path-based mode for environments without wildcard cert support - Update preflight DNS checks for both routing modes - Add Sandbox Routing Configuration section in Admin Console setup - Update requirements table to clarify routing mode dependencies Co-authored-by: openhands <openhands@all-hands.dev>
Add a warning callout explaining why subdomain-based routing is preferred from a security perspective - cookie isolation between sandboxes. Encourage customers to seek IT security exceptions. Co-authored-by: openhands <openhands@all-hands.dev>
…port Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
jpshackelford
force-pushed
the
docs/path-based-sandbox-routing
branch
from
e2ec028 to
1eccf68
Compare
|
Closing in favor of #488 - this PR had persistent merge state issues. |
4 participants
Summary
This PR updates the enterprise quick-start guide to document the path-based sandbox routing feature introduced in OpenHands-Cloud PR #563.
Background
Some enterprise deployments cannot obtain wildcard TLS certificates due to:
The path-based routing feature (released in v1.25.1) enables these deployments by routing all sandboxes through a single hostname with path-based routing instead of subdomain-based routing.
Changes
Updates to
enterprise/quick-start.mdx:Testing
The documentation follows the existing mdx format and uses standard Mintlify components (Tabs, Tab, Info, Warning).
This PR was created by an AI agent (OpenHands) on behalf of the user.