Change cloud config to only include hostname

Author: FabianKramm

pkg/devspace/cloud/config.go

@@ -22,18 +22,25 @@ type ProviderConfig map[string]*Provider
 type Provider struct {
 	Name        string `yaml:"name,omitempty"`
 	KubeContext string `yaml:"kubecontext,omitempty"`
-	Login       string `yaml:"login,omitempty"`
-	GetConfig   string `yaml:"getConfig,omitempty"`
+	Host        string `yaml:"login,omitempty"`
 	Token       string `yaml:"token,omitempty"`
 }
 
 // DevSpaceCloudProviderName is the name of the default devspace-cloud provider
 const DevSpaceCloudProviderName = "devspace-cloud"
 
+// LoginEndpoint is the cloud endpoint that will log you in
+const LoginEndpoint = "/login"
+
+// LoginSuccessEndpoint is the url redirected to after successful login
+const LoginSuccessEndpoint = "/loginSuccess"
+
+// GetClusterConfigEndpoint is the endpoint where to get the kubernetes context data
+const GetClusterConfigEndpoint = "/clusterConfig"
+
 // DevSpaceCloudProviderConfig holds the information for the devspace-cloud
 var DevSpaceCloudProviderConfig = &Provider{
-	Login:       "https://cloud.devspace.covexo.com/login",
-	GetConfig:   "https://cloud.devspace.covexo.com/clusterConfig",
+	Host:        "http://localhost:8080",
 	KubeContext: DevSpaceKubeContextName,
 }
 
@@ -58,8 +65,7 @@ func ParseCloudConfig() (ProviderConfig, error) {
 	}
 
 	if _, ok := cloudConfig[DevSpaceCloudProviderName]; ok {
-		cloudConfig[DevSpaceCloudProviderName].GetConfig = DevSpaceCloudProviderConfig.GetConfig
-		cloudConfig[DevSpaceCloudProviderName].Login = DevSpaceCloudProviderConfig.Login
+		cloudConfig[DevSpaceCloudProviderName].Host = DevSpaceCloudProviderConfig.Host
 	} else {
 		cloudConfig[DevSpaceCloudProviderName] = DevSpaceCloudProviderConfig
 	}
@@ -84,8 +90,7 @@ func SaveCloudConfig(config ProviderConfig) error {
 		provider.Name = ""
 
 		if name == DevSpaceCloudProviderName {
-			provider.Login = ""
-			provider.GetConfig = ""
+			provider.Host = ""
 		}
 	}
 

pkg/devspace/cloud/login.go

@@ -28,7 +28,7 @@ func CheckAuth(provider *Provider) (string, *api.Cluster, *api.AuthInfo, error)
 // GetClusterConfig retrieves the cluster and authconfig from the devspace cloud
 func GetClusterConfig(provider *Provider) (string, *api.Cluster, *api.AuthInfo, error) {
 	client := &http.Client{}
-	req, err := http.NewRequest("GET", provider.GetConfig, nil)
+	req, err := http.NewRequest("GET", provider.Host+GetClusterConfigEndpoint, nil)
 	if err != nil {
 		return "", nil, nil, err
 	}
@@ -83,10 +83,10 @@ func GetClusterConfig(provider *Provider) (string, *api.Cluster, *api.AuthInfo,
 func Login(provider *Provider) (string, *api.Cluster, *api.AuthInfo, error) {
 	tokenChannel := make(chan string)
 
-	log.StartWait("Logging into cloud " + provider.Login + " ...")
-	server := startServer(tokenChannel)
+	log.StartWait("Logging into cloud " + provider.Host + LoginEndpoint + " ...")
+	server := startServer(provider.Host+LoginSuccessEndpoint, tokenChannel)
 
-	open.Start(provider.Login)
+	open.Start(provider.Host + LoginEndpoint)
 
 	token := <-tokenChannel
 	close(tokenChannel)
@@ -185,19 +185,19 @@ func UpdateKubeConfig(contextName string, cluster *api.Cluster, authInfo *api.Au
 	return kubeconfig.WriteKubeConfig(config, clientcmd.RecommendedHomeFile)
 }
 
-func startServer(tokenChannel chan string) *http.Server {
+func startServer(redirectURI string, tokenChannel chan string) *http.Server {
 	srv := &http.Server{Addr: ":25853"}
 
 	http.HandleFunc("/token", func(w http.ResponseWriter, r *http.Request) {
-		fmt.Fprintf(w, "<script type=\"text/javascript\">window.close();</script>")
-
 		keys, ok := r.URL.Query()["token"]
 		if !ok || len(keys[0]) < 1 {
 			log.Fatal("Bad request")
 		}
 
 		log.StopWait()
 		tokenChannel <- keys[0]
+
+		http.Redirect(w, r, redirectURI, http.StatusSeeOther)
 	})
 
 	go func() {