You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openwhisk.apache.org by GitBox <gi...@apache.org> on 2020/04/15 03:44:40 UTC
[GitHub] [openwhisk] rabbah opened a new pull request #4882: Allow OPTIONS
response on web actions before checking for authentication.
rabbah opened a new pull request #4882: Allow OPTIONS response on web actions before checking for authentication.
URL: https://github.com/apache/openwhisk/pull/4882
## Description
This patch fixes a bug whereby webactions cannot respond to OPTIONS even for default CORS responses because the handler check for authentication first, before checking if the requests if an OPTIONS request.
Actions that opts for the default response are not affected - they are only invoked if the request is not an OPTIONS request as before. Actions which respond to OPTIONS explicitly will be invoked even if not authenticated yet but this should be acceptable since the action cannot respond to OPTIONS otherwise.
## Related issue and scope
<!--- Please include a link to a related issue if there is one. -->
- [x] I opened an issue to propose and discuss this change (#4879)
## My changes affect the following components
<!--- Select below all system components are affected by your change. -->
<!--- Enter an `x` in all applicable boxes. -->
- [ ] API
- [x] Controller
- [ ] Message Bus (e.g., Kafka)
- [ ] Loadbalancer
- [ ] Invoker
- [ ] Intrinsic actions (e.g., sequences, conductors)
- [ ] Data stores (e.g., CouchDB)
- [x] Tests
- [ ] Deployment
- [ ] CLI
- [ ] General tooling
- [ ] Documentation
## Types of changes
<!--- What types of changes does your code introduce? Use `x` in all the boxes that apply: -->
- [x] Bug fix (generally a non-breaking change which closes an issue).
- [] Enhancement or new feature (adds new functionality).
- [ ] Breaking change (a bug fix or enhancement which changes existing behavior).
## Checklist:
<!--- Please review the points below which help you make sure you've covered all aspects of the change you're making. -->
- [x] I signed an [Apache CLA](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md).
- [x] I reviewed the [style guides](https://github.com/apache/openwhisk/wiki/Contributing:-Git-guidelines#code-readiness) and followed the recommendations (Travis CI will check :).
- [x] I added tests to cover my changes.
- [x] My changes require further changes to the documentation.
- [x] I updated the documentation where necessary.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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
[GitHub] [openwhisk] ddragosd commented on issue #4882: Allow OPTIONS
response on web actions before checking for authentication.
Posted by GitBox <gi...@apache.org>.
ddragosd commented on issue #4882: Allow OPTIONS response on web actions before checking for authentication.
URL: https://github.com/apache/openwhisk/pull/4882#issuecomment-613808332
Thanks for catching this @rabbah !
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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
[GitHub] [openwhisk] codecov-io edited a comment on issue #4882: Allow
OPTIONS response on web actions before checking for authentication.
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #4882: Allow OPTIONS response on web actions before checking for authentication.
URL: https://github.com/apache/openwhisk/pull/4882#issuecomment-613814140
# [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=h1) Report
> Merging [#4882](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=desc) into [master](https://codecov.io/gh/apache/openwhisk/commit/ae2bd3a6195257847dbe45d9d4b5e5e6d2fbf6f5&el=desc) will **decrease** coverage by `6.14%`.
> The diff coverage is `88.88%`.
[](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #4882 +/- ##
==========================================
- Coverage 83.21% 77.06% -6.15%
==========================================
Files 200 200
Lines 9306 9301 -5
Branches 384 378 -6
==========================================
- Hits 7744 7168 -576
- Misses 1562 2133 +571
```
| [Impacted Files](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../apache/openwhisk/core/controller/WebActions.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb250cm9sbGVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvY29udHJvbGxlci9XZWJBY3Rpb25zLnNjYWxh) | `89.62% <88.88%> (+1.00%)` | :arrow_up: |
| [...core/database/cosmosdb/RxObservableImplicits.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvUnhPYnNlcnZhYmxlSW1wbGljaXRzLnNjYWxh) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...ore/database/cosmosdb/cache/CacheInvalidator.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0NhY2hlSW52YWxpZGF0b3Iuc2NhbGE=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...e/database/cosmosdb/cache/ChangeFeedConsumer.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0NoYW5nZUZlZWRDb25zdW1lci5zY2FsYQ==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...core/database/cosmosdb/CosmosDBArtifactStore.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvQ29zbW9zREJBcnRpZmFjdFN0b3JlLnNjYWxh) | `0.00% <0.00%> (-96.23%)` | :arrow_down: |
| [...sk/core/database/cosmosdb/CosmosDBViewMapper.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvQ29zbW9zREJWaWV3TWFwcGVyLnNjYWxh) | `0.00% <0.00%> (-93.90%)` | :arrow_down: |
| [...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0NhY2hlSW52YWxpZGF0b3JDb25maWcuc2NhbGE=) | `0.00% <0.00%> (-92.31%)` | :arrow_down: |
| [...enwhisk/connector/kafka/KamonMetricsReporter.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2Nvbm5lY3Rvci9rYWZrYS9LYW1vbk1ldHJpY3NSZXBvcnRlci5zY2FsYQ==) | `0.00% <0.00%> (-83.34%)` | :arrow_down: |
| [...e/database/cosmosdb/cache/KafkaEventProducer.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0thZmthRXZlbnRQcm9kdWNlci5zY2FsYQ==) | `0.00% <0.00%> (-77.78%)` | :arrow_down: |
| [...whisk/core/database/cosmosdb/CosmosDBSupport.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvQ29zbW9zREJTdXBwb3J0LnNjYWxh) | `0.00% <0.00%> (-74.08%)` | :arrow_down: |
| ... and [11 more](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=footer). Last update [ae2bd3a...e58400a](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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
[GitHub] [openwhisk] ddragosd commented on a change in pull request #4882:
Allow OPTIONS response on web actions before checking for authentication.
Posted by GitBox <gi...@apache.org>.
ddragosd commented on a change in pull request #4882: Allow OPTIONS response on web actions before checking for authentication.
URL: https://github.com/apache/openwhisk/pull/4882#discussion_r408572359
##########
File path: tests/src/test/scala/org/apache/openwhisk/core/controller/test/WebActionsApiTests.scala
##########
@@ -393,53 +391,97 @@ trait WebActionsApiBaseTests extends ControllerTestCommon with BeforeAndAfterEac
}
}
- it should s"reject requests when authentication is required but none given (auth? ${creds.isDefined})" in {
+ it should s"reject requests when whisk authentication is required but none given (auth? ${creds.isDefined})" in {
implicit val tid = transid()
+ val entityName = MakeName.next("export")
+ val action =
+ stubAction(
+ proxyNamespace,
+ entityName,
+ customOptions = false,
+ requireAuthentication = true,
+ requireAuthenticationAsBoolean = true)
+ val path = action.fullyQualifiedName(false)
+ put(entityStore, action)
+
allowedMethods.foreach { m =>
- Seq(true, false).foreach { useReqWhiskAuthBool =>
- requireAuthenticationAsBoolean = useReqWhiskAuthBool
+ m(s"$testRoutePath/${path}.json") ~> Route.seal(routes(creds)) ~> check {
+ if (m === Options) {
+ status should be(OK) // options response is always present regardless of auth
+ header("Access-Control-Allow-Origin").get.toString shouldBe "Access-Control-Allow-Origin: *"
+ header("Access-Control-Allow-Methods").get.toString shouldBe "Access-Control-Allow-Methods: OPTIONS, GET, DELETE, POST, PUT, HEAD, PATCH"
+ header("Access-Control-Request-Headers") shouldBe empty
+ } else if (creds.isEmpty) {
+ status should be(Unauthorized) // if user is not authenticated, reject all requests
+ } else {
+ invocationsAllowed += 1
+ status should be(OK)
+ val response = responseAs[JsObject]
+ response shouldBe JsObject(
+ "pkg" -> s"$systemId/proxy".toJson,
+ "action" -> entityName.asString.toJson,
+ "content" -> metaPayload(m.method.name.toLowerCase, JsObject.empty, creds, pkgName = "proxy"))
+ response
+ .fields("content")
+ .asJsObject
+ .fields(webApiDirectives.namespace) shouldBe creds.get.namespace.name.toJson
+ }
}
+ }
+ }
- val entityName = MakeName.next("export")
- val action = stubAction(
+ it should s"reject requests when x-authentication is required but none given (auth? ${creds.isDefined})" in {
+ implicit val tid = transid()
+
+ val entityName = MakeName.next("export")
+ val action =
+ stubAction(
proxyNamespace,
entityName,
+ customOptions = false,
requireAuthentication = true,
- requireAuthenticationAsBoolean = requireAuthenticationAsBoolean)
- val path = action.fullyQualifiedName(false)
+ requireAuthenticationAsBoolean = false)
+ val path = action.fullyQualifiedName(false)
+ put(entityStore, action)
- put(entityStore, action)
+ allowedMethods.foreach { m =>
+ // web action require-whisk-auth is set, but the header X-Require-Whisk-Auth value does not match
+ m(s"$testRoutePath/${path}.json") ~> addHeader(
+ WhiskAction.requireWhiskAuthHeader,
+ requireAuthenticationKey + "-bad") ~> Route
+ .seal(routes(creds)) ~> check {
+ if (m == Options) {
+ status should be(OK) // options should always respond
+ header("Access-Control-Allow-Origin").get.toString shouldBe "Access-Control-Allow-Origin: *"
+ header("Access-Control-Allow-Methods").get.toString shouldBe "Access-Control-Allow-Methods: OPTIONS, GET, DELETE, POST, PUT, HEAD, PATCH"
+ header("Access-Control-Request-Headers") shouldBe empty
+ } else {
+ status should be(Unauthorized)
+ }
+ }
- if (requireAuthenticationAsBoolean) {
- if (creds.isDefined) {
- val user = creds.get
- invocationsAllowed += 1
- m(s"$testRoutePath/${path}.json") ~> Route
- .seal(routes(creds)) ~> check {
- status should be(OK)
- val response = responseAs[JsObject]
- response shouldBe JsObject(
- "pkg" -> s"$systemId/proxy".toJson,
- "action" -> entityName.asString.toJson,
- "content" -> metaPayload(m.method.name.toLowerCase, JsObject.empty, creds, pkgName = "proxy"))
- response
- .fields("content")
- .asJsObject
- .fields(webApiDirectives.namespace) shouldBe user.namespace.name.toJson
- }
+ // web action require-whisk-auth is set, but the header X-Require-Whisk-Auth value is not set
+ m(s"$testRoutePath/${path}.json") ~> Route.seal(routes(creds)) ~> check {
+ if (m == Options) {
+ status should be(OK) // options should always respond
+ header("Access-Control-Allow-Origin").get.toString shouldBe "Access-Control-Allow-Origin: *"
+ header("Access-Control-Allow-Methods").get.toString shouldBe "Access-Control-Allow-Methods: OPTIONS, GET, DELETE, POST, PUT, HEAD, PATCH"
+ header("Access-Control-Request-Headers") shouldBe empty
Review comment:
👍
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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
[GitHub] [openwhisk] codecov-io commented on issue #4882: Allow OPTIONS
response on web actions before checking for authentication.
Posted by GitBox <gi...@apache.org>.
codecov-io commented on issue #4882: Allow OPTIONS response on web actions before checking for authentication.
URL: https://github.com/apache/openwhisk/pull/4882#issuecomment-613814140
# [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=h1) Report
> Merging [#4882](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=desc) into [master](https://codecov.io/gh/apache/openwhisk/commit/ae2bd3a6195257847dbe45d9d4b5e5e6d2fbf6f5&el=desc) will **decrease** coverage by `6.14%`.
> The diff coverage is `88.88%`.
[](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #4882 +/- ##
==========================================
- Coverage 83.21% 77.06% -6.15%
==========================================
Files 200 200
Lines 9306 9301 -5
Branches 384 378 -6
==========================================
- Hits 7744 7168 -576
- Misses 1562 2133 +571
```
| [Impacted Files](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../apache/openwhisk/core/controller/WebActions.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb250cm9sbGVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvY29udHJvbGxlci9XZWJBY3Rpb25zLnNjYWxh) | `89.62% <88.88%> (+1.00%)` | :arrow_up: |
| [...core/database/cosmosdb/RxObservableImplicits.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvUnhPYnNlcnZhYmxlSW1wbGljaXRzLnNjYWxh) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...ore/database/cosmosdb/cache/CacheInvalidator.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0NhY2hlSW52YWxpZGF0b3Iuc2NhbGE=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...e/database/cosmosdb/cache/ChangeFeedConsumer.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0NoYW5nZUZlZWRDb25zdW1lci5zY2FsYQ==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
| [...core/database/cosmosdb/CosmosDBArtifactStore.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvQ29zbW9zREJBcnRpZmFjdFN0b3JlLnNjYWxh) | `0.00% <0.00%> (-96.23%)` | :arrow_down: |
| [...sk/core/database/cosmosdb/CosmosDBViewMapper.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvQ29zbW9zREJWaWV3TWFwcGVyLnNjYWxh) | `0.00% <0.00%> (-93.90%)` | :arrow_down: |
| [...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0NhY2hlSW52YWxpZGF0b3JDb25maWcuc2NhbGE=) | `0.00% <0.00%> (-92.31%)` | :arrow_down: |
| [...enwhisk/connector/kafka/KamonMetricsReporter.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2Nvbm5lY3Rvci9rYWZrYS9LYW1vbk1ldHJpY3NSZXBvcnRlci5zY2FsYQ==) | `0.00% <0.00%> (-83.34%)` | :arrow_down: |
| [...e/database/cosmosdb/cache/KafkaEventProducer.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29yZS9jb3Ntb3NkYi9jYWNoZS1pbnZhbGlkYXRvci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL29wZW53aGlzay9jb3JlL2RhdGFiYXNlL2Nvc21vc2RiL2NhY2hlL0thZmthRXZlbnRQcm9kdWNlci5zY2FsYQ==) | `0.00% <0.00%> (-77.78%)` | :arrow_down: |
| [...whisk/core/database/cosmosdb/CosmosDBSupport.scala](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvY29zbW9zZGIvQ29zbW9zREJTdXBwb3J0LnNjYWxh) | `0.00% <0.00%> (-74.08%)` | :arrow_down: |
| ... and [11 more](https://codecov.io/gh/apache/openwhisk/pull/4882/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=footer). Last update [ae2bd3a...e58400a](https://codecov.io/gh/apache/openwhisk/pull/4882?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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
[GitHub] [openwhisk] rabbah merged pull request #4882: Allow OPTIONS
response on web actions before checking for authentication.
Posted by GitBox <gi...@apache.org>.
rabbah merged pull request #4882: Allow OPTIONS response on web actions before checking for authentication.
URL: https://github.com/apache/openwhisk/pull/4882
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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