fix: set correct max cost for oauth2 introspection authn handler (#1176)

ory · Aug 27, 2024 · 368c28a · 368c28a
1 parent 374b146
commit 368c28a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/.schema/config.schema.json b/.schema/config.schema.json
@@ -888,9 +888,9 @@
             },
             "max_cost": {
               "type": "integer",
-              "default": 1000,
+              "default": 100000000,
               "title": "Max Cost",
-              "description": "Max number of tokens to cache."
+              "description": "Max cost to cache."
             }
           }
         }

diff --git a/driver/configuration/provider_koanf_public_test.go b/driver/configuration/provider_koanf_public_test.go
@@ -48,7 +48,7 @@ func TestPipelineConfig(t *testing.T) {
 		p := setup(t)
 
 		require.NoError(t, p.PipelineConfig("authenticators", "oauth2_introspection", nil, &res))
-		assert.JSONEq(t, `{"cache":{"enabled":false, "max_cost":1000},"introspection_url":"https://override/path","preserve_host":false,"pre_authorization":{"client_id":"some_id","client_secret":"some_secret","enabled":true,"audience":"some_audience","scope":["foo","bar"],"token_url":"https://my-website.com/oauth2/token"},"retry":{"max_delay":"100ms", "give_up_after":"1s"},"scope_strategy":"exact"}`, string(res), "%s", res)
+		assert.JSONEq(t, `{"cache":{"enabled":false, "max_cost":100000000},"introspection_url":"https://override/path","preserve_host":false,"pre_authorization":{"client_id":"some_id","client_secret":"some_secret","enabled":true,"audience":"some_audience","scope":["foo","bar"],"token_url":"https://my-website.com/oauth2/token"},"retry":{"max_delay":"100ms", "give_up_after":"1s"},"scope_strategy":"exact"}`, string(res), "%s", res)
 	})
 
 	t.Run("case=should setup", func(t *testing.T) {

diff --git a/spec/config.schema.json b/spec/config.schema.json
@@ -904,9 +904,9 @@
             },
             "max_cost": {
               "type": "integer",
-              "default": 1000,
+              "default": 100000000,
               "title": "Max Cost",
-              "description": "Max number of tokens to cache."
+              "description": "Max cost to cache."
             }
           }
         }