Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

Use localhost instead of che-host service name for checking KC #270

Merged
merged 1 commit into from
Aug 21, 2017

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Aug 16, 2017

Make UserAuthValve and KeycloakUserChecker use localhost instead
of che-host for checkout keycloak tokens when CHE_API env var
is not set

Signed-off-by: Angel Misevski amisevsk@redhat.com

Make UserAuthValve and KeycloakUserChecker use localhost instead
of che-host for checkout keycloak tokens when CHE_API env var
is not set

Signed-off-by: Angel Misevski <amisevsk@redhat.com>
@amisevsk
Copy link
Collaborator

This is a work around for the issue of che-host not being able to communicate with its own service on minishift 1.4.1 + OS 3.6. As an alternative, this comment worked for me.

@sunix
Copy link
Contributor

sunix commented Aug 18, 2017

I don't know what's the problem and if this is related but ... I still cannot access to che-server (the server is starting) with minishift 1.4.1 with this patch
I've got this at start up of the script openshift-deploy.sh

[CHE] Creating an OS route to debug Che wsmaster...error: [the provided version "apps.openshift.io/v1" has no relevant versions: group apps.openshift.io has not been registered, no matches for apps.openshift.io/, Kind=DeploymentConfig]
See 'oc expose -h' for help and examples.

when accessing to the che-server url from the browser, i've got

504 Gateway Time-out

The server didn't respond in time.

Though these error are gone with ifconfig docker0 promisc I have other issue when using the workspace ( got .error 403 on PUT from activity.js )

selection_055

Maybe all my issue are related to keycloak + minishift 1.4.1

@benoitf benoitf merged commit 9629ed7 into redhat-developer:master Aug 21, 2017
@benoitf benoitf self-assigned this Aug 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants