fix: ignore post_logout_redirect_uris when logout is disabled (panva#…

…1221)
logto-io · May 6, 2023 · d7dd6cf · d7dd6cf
1 parent 095ca06
commit d7dd6cf
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/docs/README.md b/docs/README.md
@@ -439,6 +439,7 @@ location / {
   - [requestObjects](#featuresrequestobjects)
   - [resourceIndicators ❗](#featuresresourceindicators)
   - [revocation](#featuresrevocation)
+  - [rpInitiatedLogout](#featuresrpinitiatedlogout)
   - [userinfo](#featuresuserinfo)
 - [acrValues](#acrvalues)
 - [allowOmittingSingleRegisteredRedirectUri](#allowomittingsingleregisteredredirecturi)

diff --git a/lib/consts/client_attributes.js b/lib/consts/client_attributes.js
@@ -16,7 +16,6 @@ const RECOGNIZED_METADATA = [
   'jwks',
   'logo_uri',
   'policy_uri',
-  'post_logout_redirect_uris',
   'redirect_uris',
   'require_auth_time',
   'response_types',

diff --git a/test/configuration/client_metadata.test.js b/test/configuration/client_metadata.test.js
@@ -497,8 +497,10 @@ describe('Client metadata validation', () => {
   });
 
   context('post_logout_redirect_uris', function () {
-    defaultsTo(this.title, [], undefined);
-    defaultsTo(this.title, [], { post_logout_redirect_uris: undefined });
+    defaultsTo(this.title, []);
+    defaultsTo(this.title, undefined, undefined, {
+      features: { rpInitiatedLogout: { enabled: false } },
+    });
     mustBeArray(this.title, [{}, 'string', 123, true]);
     rejects(this.title, [123], /must only contain strings$/);