[GitHub] rabbah commented on issue #2719: wsk sdk install assumes https

[gi...@git.apache.org]

rabbah commented on issue #2719: wsk sdk install assumes https
URL: https://github.com/apache/incubator-openwhisk/issues/2719#issuecomment-329360090
 
 
   Should probably go ahead and fix all instances that prefix `https://`:
   
   ```bash
   ./go-whisk/whisk/action.go:    webActionPath := "https://%s%s/%s/web/%s/%s/%s"
   ./go-whisk/whisk/action.go:    actionPath := "https://%s%s/%s/namespaces/%s/actions/%s"
   ./go-whisk/whisk/sdk.go:    urlStr := fmt.Sprintf("https://%s/%s", s.client.Config.BaseURL.Host, relFileUrl)
   ./go-whisk-cli/commands/api.go:    api.Action.BackendUrl = "https://" + client.Config.Host + "/api/v1/web/" + qName.GetNamespace() + "/" + urlActionPackage + "/" + qName.GetEntity() + ".http"
   ./go-whisk-cli/commands/util.go:    // Can't use prettyjson util issue  https://github.com/hokaccha/go-prettyjson/issues/1 is fixed
   ./go-whisk-cli/commands/util.go:        host = "https://" + host
   ./go-whisk-cli/commands/util.go:        urlBase = fmt.Sprintf("https://%s%s", host, path)
   ```
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services