Skip to content

Commit

Permalink
refactor!: change default on allowOmittingSingleRegisteredRedirectUri
Browse files Browse the repository at this point in the history
BREAKING CHANGE: Omitting a redirect_uri parameter when a single one is registered is now enabled by default (again). This can be reverted using the `allowOmittingSingleRegisteredRedirectUri` configuration option.
  • Loading branch information
panva committed Dec 1, 2022
1 parent 0f93b8c commit d41bb0f
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -1954,7 +1954,7 @@ Allow omitting the redirect_uri parameter when only a single one is registered f

_**default value**_:
```js
false
true
```

### claims
Expand Down
2 changes: 1 addition & 1 deletion lib/helpers/defaults.js
Original file line number Diff line number Diff line change
Expand Up @@ -729,7 +729,7 @@ function makeDefaults() {
*
* title: Allow omitting the redirect_uri parameter when only a single one is registered for a client.
*/
allowOmittingSingleRegisteredRedirectUri: false,
allowOmittingSingleRegisteredRedirectUri: true,

/*
* acceptQueryParamAccessTokens
Expand Down
1 change: 0 additions & 1 deletion test/default.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ export default () => ({
],
features: {},
enabledJWA: cloneDeep({ ...JWA }),
allowOmittingSingleRegisteredRedirectUri: true,
pkce: {
required: () => false,
},
Expand Down

0 comments on commit d41bb0f

Please sign in to comment.