Introduce resource indicators#40
Conversation
doc/server/contents/conf.rst
Outdated
| "callable": validate_authorization_resource_indicators_policy, | ||
| "kwargs": { | ||
| "resource_servers_per_client": { | ||
| "CLIENT_1": { |
There was a problem hiding this comment.
The resource_indicators block is within the client configuration, so at the same level as client-id and secret. CLIENT_1 here should be the client-id. I expect that no other client-id can be defined here, and this would make CLIENT_1 redundant.
I guess the reason it is here is because the same callable is called and expects resource_servers_per_client to have the same form as it does in the global config.
doc/server/contents/conf.rst
Outdated
| }, | ||
| } | ||
|
|
||
| The policy callable accepts a specific argument list and must return the altered token exchange |
There was a problem hiding this comment.
The "token exchange" seems to be overlooked through copy-paste
ctriant
force-pushed
the
resource-indicators
branch
2 times, most recently
from
4494967 to
cfb7306
Compare
ctriant
force-pushed
the
resource-indicators
branch
3 times, most recently
from
bfec570 to
3ae0341
Compare
ctriant
force-pushed
the
resource-indicators
branch
from
3ae0341 to
b42de7b
Compare
ctriant
force-pushed
the
resource-indicators
branch
from
b42de7b to
9c81b67
Compare
Signed-off-by: Kostis Triantafyllakis <ctriant@admin.grnet.gr>
ctriant
force-pushed
the
resource-indicators
branch
from
9c81b67 to
b43e8c5
Compare
|
The checks fail and should be fixed . |
Ι think this error is carried after rebasing to the devel branch. |
|
Since it's not us but requests that has this demand I'm not sure what else we can do. |
3 participants
Signed-off-by: Kostis Triantafyllakis ctriant@admin.grnet.gr