[GitHub] csantanapr commented on issue #162: Support for overriding APIHOST url template in wskprops or cli options

[GitBox <gi...@apache.org>]

csantanapr commented on issue #162: Support for overriding APIHOST url template in wskprops or cli options
URL: https://github.com/apache/incubator-openwhisk-cli/issues/162#issuecomment-348568493
 
 
   @shazron the controller needs the /api/v1 path
   
   Can you provide more input on the use case and requirement?
   
   If you setup an edge like nginx and the CLI sends the request to `https://myserver.com/foo/bar/namespaces/_/action/myaction`
   
   Then your nginx would need to insert `/api/v1` in the middle of the url coming in anyway, and remove anything extra. so I don't get the point.
   
   IN->`https://myserver.com/foo/bar/namespaces/_/action/myaction`
   nginx will massage the URL path
   OUT to Controller -> `http://controllerhost:controllerport/api/v1/namespaces/_/action/myaction`
   
   Not saying that it can't be done I just want to understand the specific use case. 
   
   The tricky part this would be done in the go-client lower level with .wskprops
   
   I think the fix will be the same for option 1, and 2
   have a BASEPATH_V1 where by default if not specified will be `/api/v1`, or override with `/foo/bar` or `/`
   
   

----------------------------------------------------------------
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