Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable authorization directives by default #3713

Merged
merged 26 commits into from
Oct 23, 2023
Merged

Enable authorization directives by default #3713

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
17516b9
enable authorization by default
Geal Aug 31, 2023
13d63d0
missing files
Geal Aug 31, 2023
67f3ca3
extract the authenticated directive name if it changed
Geal Aug 31, 2023
c65f849
Merge branch 'dev' into geal/authz-optout
Geal Aug 31, 2023
dacae5f
fix tests for the authenticated directive
Geal Aug 31, 2023
93e4ce0
support renaming the requiresScopes directive
Geal Aug 31, 2023
cc87acc
make sure that alternative directives are not considered
Geal Aug 31, 2023
487e217
remove initial check
Geal Aug 31, 2023
f3ce2d9
update test schemas
Geal Sep 1, 2023
b4f731d
Add authorization directives support to configuration overview
Meschreiber Sep 6, 2023
229ebfa
Merge branch 'dev' into geal/authz-optout
Geal Oct 2, 2023
89140d6
split off directive renaming in a separate PR
Geal Oct 2, 2023
629d608
remove snapshot
Geal Oct 2, 2023
04962fe
use const
Geal Oct 2, 2023
740a1fa
lint
Geal Oct 2, 2023
9daae23
still check for the directive presence
Geal Oct 2, 2023
9c47a02
lint
Geal Oct 2, 2023
75b4f6d
changeset
Geal Oct 2, 2023
dd7df49
lint
Geal Oct 2, 2023
063241e
Merge branch 'dev' into geal/authz-optout
Geal Oct 6, 2023
c591020
Merge branch 'dev' into geal/authz-optout
Geal Oct 20, 2023
8d338fc
fix
Geal Oct 20, 2023
3e250c2
fix serde error
Geal Oct 20, 2023
8fa5a0d
fix again
Geal Oct 20, 2023
7cbfa2f
Merge branch 'dev' into geal/authz-optout
Geal Oct 23, 2023
6f2d1f8
Update .changesets/feat_geal_authz_optout.md
Geal Oct 23, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ...r/src/configuration/snapshots/apollo_router__configuration__tests__schema_generation.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -519,7 +519,7 @@ expression: "&schema"
"properties": {
"enabled": {
"description": "enables the `@authenticated` and `@requiresScopes` directives",
"default": false,
"default": true,
"type": "boolean"
}
}
Expand Down
Loading