Support dynamic logout URLs in CentralDashboard

[kimwnasptd]

/kind bug

We just merged for KF 1.7 RC0 the update on AuthService kubeflow/manifests#2150. But in this commit the logout URL is /authservice/logout, instead of /logout. But since we can't hardcode this value, since not everyone might be using AuthService, we'll need to have a dynamic mechanism for learning this URL.

I'd propose to

1. have a new ENV Var LOGOUT_URL in the Deployment of the app
2. The backend will add this for the /api/workgroup/env-info route handler https://github.com/kubeflow/kubeflow/blob/master/components/centraldashboard/app/api_workgroup.ts
3. The frontend will dynamically use this value from the backend

This way users can configure the app based on the authentication service that they use in a dynamic way

[kimwnasptd]

There's already an in-flight PR that tries to fix the logout behaviour #6609

We can try to fix the issue in that or create a separate one. This fix is something we'd like to track for the KF 1.7 release cc @DomFleischmann

[DomFleischmann]

Thank you for the heads up @kimwnasptd , yes I believe this is something we want to get fixed for this release.

[orfeas-k]

I sent the PR for this here.

For documentation, I followed this as a guideline in order to bind data between Polymer components. In Polymer, in order to bind values to a property of a child element, one has to use dash-case syntax instead of camel case (which is how what is being used to create the property). On top of that, in order to use a property as a DOM element's attribute, the syntax is this

attribute$={{element-property}}

[ajinkya933]

your PR routes me to localhost:8888/authservice/site/after_logout with a blank page as a response. Is there a way by which I can hardcode a website ( google.com ) to route to after logout button is clicked (provided the logout clears all cache)
cc @orfeas-k @kimwnasptd

[juliusvonkohout]

/close

since the PR is merged and oidc-authservice is gone in Kubeflow 1.9.1

[google-oss-prow]

@juliusvonkohout: Closing this issue.

In response to this:

/close

since the PR is merged and oidc-authservice is gone in Kubeflow 1.9.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.