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.

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

graphql: add GraphQL authorisation #5179

Merged
merged 25 commits into from
May 5, 2020
Merged

Conversation

MichaelJCompton
Copy link
Contributor

@MichaelJCompton MichaelJCompton commented Apr 13, 2020

Adds automated authorisation to our GraphQL endpoint.


This change is Reviewable

Docs Preview: Dgraph Preview

@@ -39,6 +39,7 @@ const (
dgraphPredArg = "pred"
idDirective = "id"
secretDirective = "secret"
authDirective = "auth"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

authDirective is unused (from varcheck)

@MichaelJCompton MichaelJCompton added the area/graphql Issues related to GraphQL support on Dgraph. label Apr 17, 2020
MichaelJCompton and others added 21 commits April 19, 2020 15:58
* Added test cases for auth schema parsing.

* Added valid schema test.

* Remove position from error.
* Send auth variable in custom jwt token. (#5220)
* graphql: auth on get and mutation results (#5259)
* Parse auth meta info from schema.
* Add support for RSA algo in JWT signing.
@MichaelJCompton MichaelJCompton marked this pull request as ready for review May 5, 2020 12:37
@MichaelJCompton MichaelJCompton requested review from manishrjain and a team as code owners May 5, 2020 12:37
@MichaelJCompton MichaelJCompton changed the title [WIP] Add GraphQL Authorisation graphql: add GraphQL authorisation May 5, 2020
@MichaelJCompton MichaelJCompton merged commit 7e1cf6a into master May 5, 2020
dna2github pushed a commit to dna2fork/dgraph that referenced this pull request Jul 18, 2020
* add Auth directive (dgraph-io#5178)
* parse auth rules (dgraph-io#5180)
* added query rewriting and e2e tests (dgraph-io#5229)
* parse and evaluate RBAC rules. (dgraph-io#5210)
* added test cases for auth schema parsing. (dgraph-io#5195)
* process auth query rules (dgraph-io#5181)
* send auth variable in custom jwt token. (dgraph-io#5220)
* auth on get and mutation results (dgraph-io#5259)
* delete authorization (dgraph-io#5270)
* parse auth meta info from schema. (dgraph-io#5269)
* auth on add update mutations (dgraph-io#5300)
* query e2e tests for authentication (dgraph-io#5312)
* more testing around additional deletes and auth (dgraph-io#5357)
* add RSA algo for JWT token verification. (dgraph-io#5358)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/graphql Issues related to GraphQL support on Dgraph.
Development

Successfully merging this pull request may close these issues.

3 participants