HTTP Permissions Refactor #4956
Conversation
stuartwdouglas
force-pushed
the
webxml-authen
branch
from
ba5a999
to
583933a
Compare
|
It may take a while for quarkus-http to hit central |
|
There is also a conflict now |
geoand
added
the
triage/needs-rebase
stuartwdouglas
force-pushed
the
webxml-authen
branch
from
583933a
to
964eff6
Compare
stuartwdouglas
removed
the
triage/needs-rebase
| * Blocking identity providers should used this context object to run blocking tasks, to prevent excessive and | ||
| * unnecessary delegation to thread pools | ||
| */ | ||
| interface AuthorizationRequestContext { |
There was a problem hiding this comment.
@stuartwdouglas Can AuthorizationRequestContext be optionally injected when needed into HttpSecurityPolicy implementations ? May be HttpAuthorizer's CONTEXT can be CDI-produced ? So that the non-blocking implementations won't see it in the signature.
Or may be simpler, just renamed to something like BlockingAuthorizationRequestContext ?
But please ignore it if you plan to expand AuthorizationRequestContext with more operations, not necessarily related to the blocking checks
There was a problem hiding this comment.
It may end up eventually containing more contextual information
|
quarkus-http Beta3 has hit Central. I rebased this one to be sure we don't have the doc issue, let's see what CI has to say. |
|
Closing. Will be part of a larger PR. |
No description provided.