graphql-dotnet / authorization

A toolset for authorizing access to graph types for GraphQL .NET.
MIT License
158 stars 38 forks source link

Additional changes for v7 compatibility #255

Closed Shane32 closed 1 year ago

Shane32 commented 1 year ago

This PR targets bump / PR #254

There's really very little code changes; mostly just refactoring.

Tests run locally but don't run because this PR doesn't target master or develop.

Hide whitespace to reduce the diff

Shane32 commented 1 year ago

@sungam3r Feel free to merge into your PR and continue to make changes as desired.

sungam3r commented 1 year ago

Thanks.

Use ExecutionOptions.User for ClaimsPrincipal if user context does not implement IProvideClaimsPrincipal

I would like to remove IProvideClaimsPrincipal at all. I see no sense to duplicate API that was added in the root lib.