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

refactor(query-graphql): implement global authorizer #1442

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

psteinroe
Copy link
Contributor

@psteinroe psteinroe commented Oct 27, 2021

first implementation for #1344

@psteinroe
Copy link
Contributor Author

@doug-martin for some reason, the e2e auth test do not run for me because the headers aren't added to the request. This is why I implemented the dirty workaround by getting the header from the rawHeaders field.

@rhessus
Copy link

rhessus commented Mar 7, 2022

Hi @steinroe, are you planning to finalize this PR ? Can I help for this ? I think this PR will solve many problems in my project due to large number of DTOs.

@psteinroe
Copy link
Contributor Author

Hey @rhessus - sorry for the late reply!

We ditched this library, so I currently do not have the capacity and need to finalise this. But as far as I remember, it was pretty close to completion and the main issues I was having was with the repo setup, because I could not run the tests properly (this also lead to the huge number of changes in this PR). I think if you are able to fix this, it should be possible to finish up. If you have any questions let me know, happy to help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants