From 13868983af76c3e82b1394a644c300db43f512d9 Mon Sep 17 00:00:00 2001 From: Salim B Date: Thu, 11 Jul 2024 14:36:49 +0200 Subject: [PATCH] fix: update OpenAPI `externalDocs` URL fixes https://github.com/PostgREST/postgrest/issues/3091 --- CHANGELOG.md | 1 + src/PostgREST/Response/OpenAPI.hs | 2 +- test/spec/Feature/OpenApi/OpenApiSpec.hs | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 61a314cb4..0aea04435 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -14,6 +14,7 @@ This project adheres to [Semantic Versioning](http://semver.org/). - #3644, Show number of timezones in schema cache load report - @wolfgangwalther - #3644, List correct enum options in OpenApi output when multiple types with same name are present - @wolfgangwalther - #3523, Fix schema cache loading retry without backoff - @steve-chavez + - #3091, Broken link in OpenAPI description `externalDocs` - @salim-b ## [12.2.1] - 2024-06-27 diff --git a/src/PostgREST/Response/OpenAPI.hs b/src/PostgREST/Response/OpenAPI.hs index d93f34236..fed13923e 100644 --- a/src/PostgREST/Response/OpenAPI.hs +++ b/src/PostgREST/Response/OpenAPI.hs @@ -402,7 +402,7 @@ postgrestSpec (prettyVersion, docsVersion) rels pds ti (s, h, p, b) sd allowSecu & description ?~ fromMaybe "This is a dynamic API generated by PostgREST" dDesc) & externalDocs ?~ ((mempty :: ExternalDocs) & description ?~ "PostgREST Documentation" - & url .~ URL ("https://postgrest.org/en/" <> docsVersion <> "/api.html")) + & url .~ URL ("https://postgrest.org/en/" <> docsVersion <> "/references/api.html")) & host .~ h' & definitions .~ fromList (makeTableDef rels <$> ti) & parameters .~ fromList (makeParamDefs ti) diff --git a/test/spec/Feature/OpenApi/OpenApiSpec.hs b/test/spec/Feature/OpenApi/OpenApiSpec.hs index 0fc29508b..2a0f49ace 100644 --- a/test/spec/Feature/OpenApi/OpenApiSpec.hs +++ b/test/spec/Feature/OpenApi/OpenApiSpec.hs @@ -45,7 +45,7 @@ spec actualPgVersion = describe "OpenAPI" $ do let docsUrl = r ^? key "externalDocs" . key "url" - liftIO $ docsUrl `shouldBe` Just (String ("https://postgrest.org/en/" <> docsVersion <> "/api.html")) + liftIO $ docsUrl `shouldBe` Just (String ("https://postgrest.org/en/" <> docsVersion <> "/references/api.html")) describe "schema" $ do