[GitHub] csantanapr commented on issue #12: Swift SDK does not support self-signed server certificates.

2018-02-03 Thread GitBox
csantanapr commented on issue #12: Swift SDK does not support self-signed server certificates. URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/12#issuecomment-362800244 Yep that correct it would no be the default. Default behavior is that user coding the action

[GitHub] jthomas commented on issue #12: Swift SDK does not support self-signed server certificates.

2018-02-03 Thread GitBox
jthomas commented on issue #12: Swift SDK does not support self-signed server certificates. URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/12#issuecomment-362798918 In the code above, this won't be the default behaviour. It relies on the user explictly setting a

[GitHub] csantanapr commented on issue #681: proper handling of 'default' package name

2018-02-03 Thread GitBox
csantanapr commented on issue #681: proper handling of 'default' package name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/681#issuecomment-362800759 I also think agree with @lionelvillard let?s allow to be at the root for actions, rules, triggers

[GitHub] dgrove-oss commented on issue #3245: bypassPullForLocalImages setting no longer overridable

2018-02-03 Thread GitBox
dgrove-oss commented on issue #3245: bypassPullForLocalImages setting no longer overridable URL: https://github.com/apache/incubator-openwhisk/issues/3245#issuecomment-362820844 If you want to assign this to me, I can put a fix together on Monday.

[GitHub] csantanapr commented on issue #15: Do not use own copy of actionProxy.py use common one from dockerskeleton

2018-02-03 Thread GitBox
csantanapr commented on issue #15: Do not use own copy of actionProxy.py use common one from dockerskeleton URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/15#issuecomment-362854034 Depends on https://github.com/apache/incubator-openwhisk-runtime-docker/issues/13

[GitHub] csantanapr opened a new issue #16: Error with urlSession double free or corruption (fasttop) upgrade to swift 4.0.3

2018-02-03 Thread GitBox
csantanapr opened a new issue #16: Error with urlSession double free or corruption (fasttop) upgrade to swift 4.0.3 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/16 Doing a non blocking invoke using urlSession it fails with ``` Starting test Swift Whisk SDK

[GitHub] csantanapr closed issue #13: Update tests cases to allow to take gradle to override the use of https for Swift 4

2018-02-03 Thread GitBox
csantanapr closed issue #13: Update tests cases to allow to take gradle to override the use of https for Swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/13 This is an automated message from

[GitHub] dubeejw commented on a change in pull request #17: Various fixes for swift 4

2018-02-03 Thread GitBox
dubeejw commented on a change in pull request #17: Various fixes for swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/17#discussion_r165835866 ## File path: tests/src/test/scala/actionContainers/Swift4ActionContainerTests.scala ## @@ -91,4

[GitHub] csantanapr opened a new pull request #17: Various fixes for swift 4

2018-02-03 Thread GitBox
csantanapr opened a new pull request #17: Various fixes for swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/17 Fixes #16 with urlSesson double free or corruption Fixes #14 with large paramters

[GitHub] csantanapr closed issue #6: Refactor Whisk class for swift4

2018-02-03 Thread GitBox
csantanapr closed issue #6: Refactor Whisk class for swift4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/6 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] csantanapr commented on a change in pull request #17: Various fixes for swift 4

2018-02-03 Thread GitBox
csantanapr commented on a change in pull request #17: Various fixes for swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/17#discussion_r165836114 ## File path: tests/src/test/scala/actionContainers/Swift4ActionContainerTests.scala ## @@ -91,4

[GitHub] csantanapr commented on issue #10: Remove redundant file names from exclusion list.

2018-02-03 Thread GitBox
csantanapr commented on issue #10: Remove redundant file names from exclusion list. URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/10#issuecomment-362885999 Interesting use case @jthomas We have not documented, or supported this use case, of having a source file

[GitHub] csantanapr commented on issue #6: Refactor Whisk class for swift4

2018-02-03 Thread GitBox
csantanapr commented on issue #6: Refactor Whisk class for swift4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/6#issuecomment-362884638 We are using urlSession now

[GitHub] csantanapr commented on issue #17: Various fixes for swift 4

2018-02-03 Thread GitBox
csantanapr commented on issue #17: Various fixes for swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/17#issuecomment-362884693 FYI @jthomas This is an automated message from the Apache Git

[GitHub] dubeejw commented on a change in pull request #17: Various fixes for swift 4

2018-02-03 Thread GitBox
dubeejw commented on a change in pull request #17: Various fixes for swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/17#discussion_r165835866 ## File path: tests/src/test/scala/actionContainers/Swift4ActionContainerTests.scala ## @@ -91,4

[GitHub] dubeejw commented on a change in pull request #17: Various fixes for swift 4

2018-02-03 Thread GitBox
dubeejw commented on a change in pull request #17: Various fixes for swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/17#discussion_r165835822 ## File path: tests/src/test/scala/actionContainers/Swift4ActionContainerTests.scala ## @@ -91,4

[GitHub] dubeejw commented on a change in pull request #17: Various fixes for swift 4

2018-02-03 Thread GitBox
dubeejw commented on a change in pull request #17: Various fixes for swift 4 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/17#discussion_r165835829 ## File path: tests/dat/build.sh ## @@ -6,4 +6,6 @@ set -e ../../tools/build/compile.sh

[GitHub] csantanapr opened a new issue #15: Do not use own copy of actionProxy.py use common one from dockerskeleton

2018-02-03 Thread GitBox
csantanapr opened a new issue #15: Do not use own copy of actionProxy.py use common one from dockerskeleton URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/15 We currently have our own copy of