@defra/lis-infra-ui-services 0.39.0 → 0.40.0

package/README.md

@@ -177,9 +177,10 @@ The main entry point (`@defra/lis-infra-ui-services`) exports `buildMicrositePat
 | ------------- | ---------------------- |
 | 404           | `page-not-found`       |
 | 401, 403      | `not-authorised`       |
-| 503           | `service-unavailable`  |
 | anything else | `problem-with-service` |
 
+A 503 from a dependency (a service token that cannot be issued, an open circuit) is a problem with the service, not maintenance. The `service-unavailable` page is only rendered by the maintenance plugin below, which knows when the service is back.
+
 5xx responses are logged at `error`, and 4xx responses other than 401, 403 and 404 at `warn`, each with the error object and the ECS fields `http.request.method`, `url.path` and `http.response.status_code`. Boom messages are logged, so application code must not put user-supplied text in them.
 
 ```js

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra/lis-infra-ui-services",
-  "version": "0.39.0",
+  "version": "0.40.0",
   "description": "UI-Services shared definitions for the livestock taxonomy topology.",
   "main": "src/index.js",
   "repository": {

package/src/errors/index.js

@@ -5,8 +5,7 @@ import { isAnsweredByMaintenance } from '../maintenance.js'
 import {
   notAuthorised,
   pageNotFound,
-  problemWithService,
-  serviceUnavailable
+  problemWithService
 } from './error-pages.js'
 import { statusCodes } from '../status-codes.js'
 
@@ -17,8 +16,6 @@ function errorPageFor(statusCode) {
     case statusCodes.unauthorized:
     case statusCodes.forbidden:
       return notAuthorised
-    case statusCodes.serviceUnavailable:
-      return serviceUnavailable
     default:
       return problemWithService
   }

package/src/errors/index.test.js

@@ -67,8 +67,8 @@ test.each([
   ],
   [
     statusCodes.serviceUnavailable,
-    'error/service-unavailable',
-    'Sorry, the service is unavailable'
+    'error/problem-with-service',
+    'Sorry, there is a problem with the service'
   ]
 ])(
   'catchAll renders a %i as the %s page titled "%s"',