Skip to content

fix(abs): show Temporarily Unavailable on 5xx instead of Record Not Found#800

Open
thostetler wants to merge 4 commits intoadsabs:masterfrom
thostetler:fix/abs-service-unavailable
Open

fix(abs): show Temporarily Unavailable on 5xx instead of Record Not Found#800
thostetler wants to merge 4 commits intoadsabs:masterfrom
thostetler:fix/abs-service-unavailable

Conversation

@thostetler
Copy link
Member

When the upstream search API returns a 5xx during SSR for an abstract
page, users see "Record Not Found" — the same message as a genuinely
missing record. This is misleading; the record exists but the service
was temporarily unavailable.

  • Added statusCode to SSR error props in absCanonicalization.ts
  • Created ServiceUnavailable component with retry/back actions
  • Updated all 10 abstract pages to show ServiceUnavailable for 5xx
  • RecordNotFound still shown for actual missing records (no statusCode or 4xx)

@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 94.36620% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.6%. Comparing base (968d8a6) to head (12eb257).

Files with missing lines Patch % Lines
src/lib/serverside/absCanonicalization.ts 25.0% 3 Missing ⚠️
...mponents/ServiceUnavailable/ServiceUnavailable.tsx 98.6% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #800     +/-   ##
========================================
+ Coverage    61.6%   61.6%   +0.1%     
========================================
  Files         312     313      +1     
  Lines       36000   36070     +70     
  Branches     1594    1597      +3     
========================================
+ Hits        22148   22213     +65     
- Misses      13815   13820      +5     
  Partials       37      37             
Files with missing lines Coverage Δ
...mponents/ServiceUnavailable/ServiceUnavailable.tsx 98.6% <98.6%> (ø)
src/lib/serverside/absCanonicalization.ts 62.7% <25.0%> (-0.4%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@thostetler thostetler marked this pull request as ready for review February 13, 2026 03:31
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