Add middleware to pull out auth state from the JWT. #183
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
|
||
|
|
||
| def create_app() -> Quart: | ||
| app: Quart = Quart(__name__) | ||
| app.request_class = UmaAuthRequest |
There was a problem hiding this comment.
@yunyuyunyu let me know if there's a more correct way to do this. I feel like I'm missing something because I still need to cast to this type in wrappers.py. Not sure if I'm actually getting the benefit of this line as a result.
There was a problem hiding this comment.
If you don't set it, in require_auth the request you are getting most likely will not be a UmaAuthRequest
There was a problem hiding this comment.
It does seem to be working for me in unit tests and when running locally, so I think this is right?
jklein24
changed the title
|
|
||
|
|
||
| def create_app() -> Quart: | ||
| app: Quart = Quart(__name__) | ||
| app.request_class = UmaAuthRequest |
There was a problem hiding this comment.
If you don't set it, in require_auth the request you are getting most likely will not be a UmaAuthRequest
jklein24
force-pushed
the
feat/authmiddle
branch
from
b5b0307
to
7ad8496
Compare
jklein24
force-pushed
the
feat/authmiddle
branch
from
7ad8496
to
41205af
Compare
jklein24
force-pushed
the
feat/authmiddle
branch
from
23fc389
to
9de4c2b
Compare
Also refactoring the frontend's API to a blueprint for better modularization in the process.