Skip to content

fix: resolve redirect chain causing 404s for system-overview pages #716

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Younique98
Copy link
Contributor

@Younique98 Younique98 commented Aug 11, 2025

PR Description

Problem

Several Terraform Enterprise system-overview pages were returning 404 errors in production:

  • /terraform/enterprise/system-overview/reliability-availability
  • /terraform/enterprise/system-overview/capacity
  • /terraform/enterprise/system-overview/security-model
  • /terraform/enterprise/system-overview/data-security

Root Cause

The redirects created a chain that failed in production environments. For example, lines 480-485 and the redirect pattern around line 590+ created this problematic chain:

/terraform/enterprise/system-overview/reliability-availability
    ↓ (lines 480-485: first redirect)
/terraform/enterprise/replicated/architecture/system-overview/reliability-availability
    ↓ (line ~590: second redirect pattern)
/terraform/enterprise/deploy/replicated/architecture/system-overview/reliability-availability

Production environments handle redirect chains differently than development/preview environments, causing the chain to break and return 404s.

Screen Recording:

realiability-and-availablity-capacity-screenrecording-no-audio.mov
reliability-and-availability-screen-recording-no-audio.mov

Solution

Updated the redirects in redirects.jsonc to point directly to the final destinations, eliminating the redirect chain.

Before (line 480-485):

{
  "source": "/terraform/enterprise/system-overview/reliability-availability",
  "destination": "/terraform/enterprise/replicated/architecture/system-overview/reliability-availability",
  "permanent": true
}

After (line 480-485):

{
  "source": "/terraform/enterprise/system-overview/reliability-availability",  
  "destination": "/terraform/enterprise/deploy/replicated/architecture/system-overview/reliability-availability",
  "permanent": true
}

Files Changed

  • redirects.jsonc (lines 480-503)

Testing

  • Verified locally that redirects work as expected
  • Confirmed direct navigation to final URLs works in both environments
  • Applied same pattern to all affected system-overview pages

Impact

This resolves the 404 errors for users accessing these system-overview pages in production while maintaining backward compatibility for bookmarked URLs.

Update system-overview redirects to point directly to final destinations
instead of creating redirect chains that fail in production environments.
@Younique98 Younique98 requested review from a team as code owners August 11, 2025 23:54
Copy link

github-actions bot commented Aug 11, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Tue Aug 12 00:22:53 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Tue Aug 12 00:10:46 UTC 2025

Copy link

Broken Link Checker

Full Github Actions output

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.

1 participant