-
Notifications
You must be signed in to change notification settings - Fork 357
Support for filter on webhook json response body #3444
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
Comments
Hello, If you have only a single service guid (e.g. when running skipper as auth sidecar for a microservice) then maybe it could be possible to configure |
Thanks a lot for your prompt answer! My use case requires verifying permission for a single single guid so your proposal sounds great. How hard would be the type conversion from boolean to string ? Is it something that could fit a release soon ? |
Sorry, the guid to check is indeed dynamic, and I need to use the filter within the k8s ingress controller, so as you said this would require a "new tokeninfo family of filters that accept tokeninfo url as an argument per filter instance" |
Is your feature request related to a problem? Please describe.
Need to leverage a 3rd party authorization endpoint which returns the authZ decison in json body (and not as http status/headers)
Describe the solution you would like
a new filter which would combine
Describe alternatives you've considered (optional)
Using existing filters
Additional context (optional)
Some 3rd party systems provide authorization decisions as Json documents instead of HTTP headers.
The cloudfoundry platform as a service, relies on such mechanism, where it exposes an api endpoint
/v3/service_instances/:guid/permission
to verify whether the current browser session is authenticated and authorized to perform two static actions: read and manage. This endpoint returns a json payload such asSee more details in https://docs.cloudfoundry.org/services/dashboard-sso.html#checking-user-permissions
Would you like to work on it?
Yes, but no golang skills and little time
The text was updated successfully, but these errors were encountered: